https://bugs.winehq.org/show_bug.cgi?id=53207
Bug ID: 53207 Summary: user32:msg - test_button_messages() fails randomly in Wine Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: user32 Assignee: wine-bugs@winehq.org Reporter: fgouget@codeweavers.com Distribution: ---
user32:msg - test_button_messages() fails randomly in Wine. There are two failure modes:
* WM_CTLCOLORBTN (0x135) + WM_DRAWITEM (0x2b) -> 6 failures msg.c:6881: Test failed: SetFocus(hwnd) on a button: 4: the msg 0x0007 was expected, but got msg 0x0005 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 5: the msg 0x0135 was expected, but got msg 0x030f instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 6: the msg 0x002b was expected, but got msg 0x001c instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 7: the msg 0x0111 was expected, but got msg 0x0086 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 8: the msg 0x8000 was expected, but got msg 0x0006 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 9: the msg sequence is not complete: expected 0000 - actual 0009
* WM_CTLCOLORSTATIC (0x138) -> 5 failures msg.c:6881: Test failed: SetFocus(hwnd) on a button: 4: the msg 0x0007 was expected, but got msg 0x0005 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 5: the msg 0x0138 was expected, but got msg 0x030f instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 6: the msg 0x0111 was expected, but got msg 0x001c instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 8: the msg 0x8000 was expected, but got msg 0x0086 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 9: the msg sequence is not complete: expected 0000 - actual 0006
https://test.winehq.org/data/patterns.html#user32:msg
These failures don't seem to depend on the locale or bitness and were first seen on 2022-06-06.
https://bugs.winehq.org/show_bug.cgi?id=53207
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |source, testcase
https://bugs.winehq.org/show_bug.cgi?id=53207
--- Comment #1 from François Gouget fgouget@codeweavers.com --- There is another 'WM_CTLCOLORBTN' failure mode:
* WM_CTLCOLORBTN (0x135) + WM_DRAWITEM (0x2b) -> 6 failures msg.c:6881: Test failed: SetFocus(hwnd) on a button: 3: the msg 0x8005 was expected, but got msg 0x0005 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 4: the msg 0x0007 was expected, but got msg 0x0003 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 5: the msg 0x0135 was expected, but got msg 0x0046 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 6: the msg 0x002b was expected, but got msg 0x0047 instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 7: the msg 0x0111 was expected, but got msg 0x001c instead msg.c:6881: Test failed: SetFocus(hwnd) on a button: 8: the msg 0x8000 was expected, but got msg 0x0086 instead
https://bugs.winehq.org/show_bug.cgi?id=53207
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|user32:msg - |user32:msg - |test_button_messages() |test_button_messages() |fails randomly in Wine |fails randomly in Windows | |and Wine
--- Comment #2 from François Gouget fgouget@codeweavers.com --- These failures actually also happen on Windows.
https://bugs.winehq.org/show_bug.cgi?id=53207
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |cb0e4f5f4f7dc10a0933349ad78 | |0a11405cd736e Status|NEW |RESOLVED
--- Comment #3 from François Gouget fgouget@codeweavers.com --- This got fixed by the commit below:
commit cb0e4f5f4f7dc10a0933349ad780a11405cd736e Author: Akihiro Sagawa sagawa.aki@gmail.com Date: Mon Sep 19 22:10:24 2022 +0900
user32: Fix a memory allocation strategy.
Otherwise, the function allocates a heap memory when prev_size is enough. What is worse is that it returns the buffer untouched if the prev_size is insufficient.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53703
https://bugs.winehq.org/show_bug.cgi?id=53207
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #4 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 8.0-rc1.