[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: wine-bugs(a)winehq.org Reporter: austinenglish(a)gmail.com 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== -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38462 --- Comment #1 from Nikolay Sivov <bunglehead(a)gmail.com> --- 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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38462 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jacek(a)codeweavers.com --- Comment #2 from Jacek Caban <jacek(a)codeweavers.com> --- JScript does weird thing in this case: http://source.winehq.org/patches/data/111606 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38462 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|mshtml |jscript -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38462 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |ccccd1c43576943f958ee76b654 | |f2f9c0aedf06b Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #3 from Jacek Caban <jacek(a)codeweavers.com> --- Fixed in Git. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38462 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.7.45. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs@winehq.org