[Bug 38462] New: valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

Bug ID: 38462
           Summary: valgrind shows an uninitialized variable in
                    dlls/mshtml/tests/activex.c test_exec_script()
           Product: Wine
           Version: 1.7.41
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: mshtml
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

==10009== Conditional jump or move depends on uninitialised value(s)
==10009==    at 0x4E8752B: VARIANT_ValidateType (variant.c:512)
==10009==    by 0x4E878FD: VariantClear (variant.c:626)
==10009==    by 0x487C37B: test_exec_script (activex.c:2619)
==10009==    by 0x487C468: test_create_element (activex.c:2641)
==10009==    by 0x487CE41: func_activex (activex.c:2807)
==10009==    by 0x4900DDF: run_test (test.h:584)
==10009==    by 0x4901219: main (test.h:666)
==10009==  Uninitialised value was created by a stack allocation
==10009==    at 0x487C26B: test_exec_script (activex.c:2604)
==10009==

[Bug 38462] valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

--- Comment #1 from Nikolay Sivov  ---
What happens is that execScript() doesn't touch returned variant at all;
jscript will set it only if SCRIPTTEXT_ISEXPRESSION is specified, which is not
a case here. I tried with NULL argument and it uncovered another problem - wine
accepts it while execScript() fails on windows.

This needs more tests to see what execScript() returns in this variant.

[Bug 38462] valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Jacek Caban  ---
JScript does weird thing in this case:

http://source.winehq.org/patches/data/111606

[Bug 38462] valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|mshtml                      |jscript

[Bug 38462] valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |ccccd1c43576943f958ee76b654
                   |                            |f2f9c0aedf06b
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Jacek Caban  ---
Fixed in Git.

[Bug 38462] valgrind shows an uninitialized variable in dlls/mshtml/tests/activex.c test_exec_script()

https://bugs.winehq.org/show_bug.cgi?id=38462

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.45.