[Bug 36809] New: valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

Bug ID: 36809
           Summary: valgrind shows an unitialized variable in
                    dlls/dmscript/tests/dmscript.c
           Product: Wine
           Version: 1.7.20
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: directx-dmusic
          Assignee: [e-mail]
          Reporter: [e-mail]

==20119== Use of uninitialised value of size 4
==20119==    at 0x427A0CD: _itoa_word (_itoa.c:180)
==20119==    by 0x427D856: vfprintf (vfprintf.c:1634)
==20119==    by 0x429E4E1: vsprintf (iovsprintf.c:42)
==20119==    by 0x428429E: sprintf (sprintf.c:32)
==20119==    by 0x4BC691C: wine_dbgstr_guid (test.h:533)
==20119==    by 0x4BC5B0F: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x427A0D3: _itoa_word (_itoa.c:180)
==20119==    by 0x427D856: vfprintf (vfprintf.c:1634)
==20119==    by 0x429E4E1: vsprintf (iovsprintf.c:42)
==20119==    by 0x428429E: sprintf (sprintf.c:32)
==20119==    by 0x4BC691C: wine_dbgstr_guid (test.h:533)
==20119==    by 0x4BC5B0F: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x427ADD7: vfprintf (vfprintf.c:1634)
==20119==    by 0x429E4E1: vsprintf (iovsprintf.c:42)
==20119==    by 0x428429E: sprintf (sprintf.c:32)
==20119==    by 0x4BC691C: wine_dbgstr_guid (test.h:533)
==20119==    by 0x4BC5B0F: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x427AE43: vfprintf (vfprintf.c:1634)
==20119==    by 0x429E4E1: vsprintf (iovsprintf.c:42)
==20119==    by 0x428429E: sprintf (sprintf.c:32)
==20119==    by 0x4BC691C: wine_dbgstr_guid (test.h:533)
==20119==    by 0x4BC5B0F: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x427AEC5: vfprintf (vfprintf.c:1634)
==20119==    by 0x429E4E1: vsprintf (iovsprintf.c:42)
==20119==    by 0x428429E: sprintf (sprintf.c:32)
==20119==    by 0x4BC691C: wine_dbgstr_guid (test.h:533)
==20119==    by 0x4BC5B0F: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x402EB3D: bcmp (vg_replace_strmem.c:965)
==20119==    by 0x4BC5B2E: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

==20119== Conditional jump or move depends on uninitialised value(s)
==20119==    at 0x4BC5F76: winetest_vok (test.h:311)
==20119==    by 0x4BC6184: winetest_ok (test.h:357)
==20119==    by 0x4BC5B4C: test_scripttrack (dmscript.c:222)
==20119==    by 0x4BC5D40: func_dmscript (dmscript.c:249)
==20119==    by 0x4BC6B3B: run_test (test.h:584)
==20119==    by 0x4BC6F29: main (test.h:654)
==20119==  Uninitialised value was created by a stack allocation
==20119==    at 0x4BC597E: test_scripttrack (dmscript.c:206)
==20119==

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

--- Comment #1 from Bruno Jesus  ---
Created attachment 49278
  --> https://bugs.winehq.org/attachment.cgi?id=49278
patch

Please try the attached patch.

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

--- Comment #2 from Austin English  ---
(In reply to Bruno Jesus from comment #1)
> Created attachment 49278 [details]
> patch
>
> Please try the attached patch.

Works here, thanks.

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

--- Comment #3 from Nikolay Sivov  ---
Patch sent
https://www.winehq.org/pipermail/wine-patches/2015-April/138508.html.

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |52251ac6da29c261501029c8868
                   |                            |368c37840a69b
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Nikolay Sivov  ---
Should be fixed with 52251ac6da29c261501029c8868368c37840a69b.

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

--- Comment #5 from Austin English  ---
(In reply to Nikolay Sivov from comment #4)
> Should be fixed with 52251ac6da29c261501029c8868368c37840a69b.

Yep, thanks.

[Bug 36809] valgrind shows an unitialized variable in dlls/dmscript/tests/dmscript.c

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

Alexandre Julliard  changed:

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

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.41.