[Bug 36185] New: comctl32/button tests shows an unitialized variable under valgrind

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

            Bug ID: 36185
           Summary: comctl32/button tests shows an unitialized variable
                    under valgrind
           Product: Wine
           Version: 1.3.13
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: minor
          Priority: P2
         Component: comctl32
          Assignee: wine-bugs(a)winehq.org
          Reporter: austinenglish(a)gmail.com
                CC: dank(a)kegel.com

==2251== Conditional jump or move depends on uninitialised value(s)
==2251==    at 0x4F10DEC: winetest_vok (test.h:311)
==2251==    by 0x4F10FFA: winetest_ok (test.h:357)
==2251==    by 0x4E65D82: test_button_class (button.c:748)
==2251==    by 0x4E6628D: func_button (button.c:810)
==2251==    by 0x4F119B1: run_test (test.h:584)
==2251==    by 0x4F11DA0: main (test.h:654)
==2251==  Uninitialised value was created by a stack allocation
==2251==    at 0x4E65A95: test_button_class (button.c:722)
==2251== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:winetest_vok
   fun:winetest_ok
   fun:test_button_class
   fun:func_button
   fun:run_test
   fun:main
}

looking at the code, it's semi-expected, as it's a todo_wine:
    ret = GetClassInfoExW(NULL, nameW, &ex2W);
todo_wine {
    ok(ret, "got %d\n", ret);
    ok(ex2W.lpfnWndProc == exW.lpfnWndProc, "got %p, %p\n", exW.lpfnWndProc,
ex2W.lpfnWndProc);
}

-- 
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.

[Bug 36185] comctl32/button tests shows an unitialized variable under valgrind

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

Austin English <austinenglish(a)gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |valgrind

-- 
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.

[Bug 36185] comctl32/button tests shows an unitialized variable under valgrind

http://bugs.winehq.org/show_bug.cgi?id=36185

--- Comment #1 from Nikolay Sivov <bunglehead(a)gmail.com> ---
Sent a patch
http://www.winehq.org/pipermail/wine-patches/2014-June/132978.html.

-- 
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.

[Bug 36185] comctl32/button tests shows an unitialized variable under valgrind

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

Austin English <austinenglish(a)gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |055129485d022976caeacbdf4e9
                   |                            |8f982c88e9d07
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Austin English <austinenglish(a)gmail.com> ---
https://source.winehq.org/git/wine.git/commitdiff/055129485d022976caeacbdf4e...

-- 
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.

[Bug 36185] comctl32/button tests shows an unitialized variable under valgrind

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

Alexandre Julliard <julliard(a)winehq.org> changed:

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

--- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> ---
Closing bugs fixed in 1.7.21.

-- 
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.