flaky uninit variable in jscript

[Dan Kegel]

There was a second new error in that file that is probably low hanging fruit:
 Conditional jump or move depends on uninitialised value(s)
    at  invoke_constructor (function.c:243)
    by  Function_value (function.c:521)
    by  invoke_prop_func (dispex.c:259)
    by  jsdisp_call (dispex.c:870)
    by  disp_call (dispex.c:899)
    by  new_expression_eval (engine.c:1568)
    by  expr_eval (engine.c:42)
    by  assign_expression_eval (engine.c:3110)
    by  expr_eval (engine.c:42)
    by  expression_statement_eval (engine.c:679)
    by  stat_eval (engine.c:37)
    by  exec_source (engine.c:473)
    by  exec_global_code (jscript.c:107)
    by  JScriptParse_ParseScriptText (jscript.c:717)
    by  parse_script (run.c:765)
    by  run_from_res (run.c:908)
    by  run_tests (run.c:1057)
    by  func_run (run.c:1095)
    by  run_test (test.h:535)
    by  main (test.h:585)
  Uninitialised value was created by a stack allocation
    at  invoke_constructor (function.c:227)

Both are still happening as of the last run I have,
http://kegel.com/wine/valgrind/logs/2009-11-17-07.31/vg-jscript_run.txt
- Dan

On Thu, Nov 5, 2009 at 7:25 PM, Dan Kegel <dank at kegel.com> wrote:
> Hi Jacek,
> could you have a look at this?  I think there are some variants that
> aren't being
> cleanly initialized.
>
> http://kegel.com/wine/valgrind/logs/2009-11-05-09.09/diff-jscript_run.txt
> http://kegel.com/wine/valgrind/logs/2009-11-05-09.09/vg-jscript_run.txt
>
>  Conditional jump or move depends on uninitialised value(s)
>    at  VARIANT_ValidateType (variant.c:546)
>    by  VariantClear (variant.c:599)
>    by  exec_global_code (jscript.c:111)
>    by  JScriptParse_ParseScriptText (jscript.c:717)
>    by  parse_text (script.c:624)
>    by  parse_inline_script (script.c:679)
>    by  parse_script_elem (script.c:703)
>    by  doc_insert_script (script.c:817)
>    by  nsRunnable_Run (mutation.c:464)
>    by  ???
>    by  ???
>    by  add_script_runner (mutation.c:196)
>    by  push_mutation_queue (mutation.c:253)
>    by  nsDocumentObserver_DoneAddingChildren (mutation.c:684)
>    by  ???
>    by  ???
>    by  ???
>    by  ???
>    by  ???
>    by  ???
>  Uninitialised value was created by a stack allocation
>    at  exec_global_code (jscript.c:94)
>
> Thanks,
> Dan
>