[Bug 59086] New: ChickenTournament crashes since the user32 control copies were removed from comctl32 v5
http://bugs.winehq.org/show_bug.cgi?id=59086 Bug ID: 59086 Summary: ChickenTournament crashes since the user32 control copies were removed from comctl32 v5 Product: Wine Version: 10.17 Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: comctl32 Assignee: wine-bugs(a)list.winehq.org Reporter: mstefani(a)winehq.org Distribution: --- Created attachment 79873 --> http://bugs.winehq.org/attachment.cgi?id=79873 Log of WINEDEBUG='+seh,+button,+combo,+commctrl,+edit,+listbox,+static' ChickenTournament demo crashes since # first bad commit: [a48455c735544fc53696ef7ec84f1da0ae8e7c74] comctl32: Remove user32 control copies in comctl32 v5. It works just fine with the previous commit. I've grabbed the debug log for the debug channels in the affected files of that commit but there isn't much, except tons of trace:seh: which I see too when the program works WINEDEBUG='+seh,+button,+combo,+commctrl,+edit,+listbox,+static' 0024:trace:commctrl:DllMain 76160000, 0x1, 0031FD24 0024:trace:commctrl:DllMain Subclassing atom added: 0000C017 0024:trace:commctrl:DllMain 76160000, 0, 00000001 Full log in the attachment. SHA1: 894aa6fc681a385822d50a9f8a4b3cfac62db06f ct.zip SHA256: db7e728167e694e6fea3fa9858f457ec2c4d076c25cc67225f0a976603eeb240 ct.zip The demo is still available to download and the source code is on github. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- URL| |http://www.saschahlusiak.de | |/chicken-tournament/ Keywords| |download, regression, | |source Regression SHA1| |a48455c735544fc53696ef7ec84 | |f1da0ae8e7c74 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|ChickenTournament crashes |ChickenTournament doesn't |since the user32 control |starts since the user32 |copies were removed from |control copies were removed |comctl32 v5 |from comctl32 v5 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Zeb Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zzhang(a)codeweavers.com -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 --- Comment #1 from Zhiyi Zhang <zzhang(a)codeweavers.com> --- I will take a look. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Zhiyi Zhang <zzhang(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|wine-bugs(a)list.winehq.org |zzhang(a)codeweavers.com -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Zhiyi Zhang <zzhang@codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1|a48455c735544fc53696ef7ec84 |7abf4432a7dfdc044f0458f2457 |f1da0ae8e7c74 |829038dc497f0 --- Comment #2 from Zhiyi Zhang <zzhang@codeweavers.com> --- The bad commit is from the following commit on my machine ``` 7abf4432a7dfdc044f0458f2457829038dc497f0 is the first bad commit commit 7abf4432a7dfdc044f0458f2457829038dc497f0 Author: Rémi Bernon <rbernon@codeweavers.com> Date: Tue Sep 16 10:02:09 2025 +0200 winex11: Use EGL by default for OpenGL rendering. Notes: Approved-by: Alexandre Julliard (@julliard) Merge-Request: https://gitlab.winehq.org/wine/wine/merge_requests/8977 dlls/winex11.drv/opengl.c | 5 ++--- dlls/winex11.drv/x11drv_main.c | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) ``` However, the real issue is an OOM error, which explains why you bisected to a different commit. Fortunately, Rémi's https://gitlab.winehq.org/wine/wine/-/merge_requests/9736 fixes this. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Zhiyi Zhang <zzhang@codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|zzhang@codeweavers.com |wine-bugs@list.winehq.org -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Michael Stefaniuc <mstefani@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1|7abf4432a7dfdc044f0458f2457 |a48455c735544fc53696ef7ec84 |829038dc497f0 |f1da0ae8e7c74 --- Comment #3 from Michael Stefaniuc <mstefani@winehq.org> --- I never ever experienced that EGL OOM. And none (not just the Chicken Tournament one) of my logs show the error from #59093. err:wgl:X11DRV_WineGL_InitOpenglInfo Maybe because I am using 32bit bottles aka WINEARCH=win32. There is a reason I put in my initial report "It works just fine with the previous commit", I always test the previous commit from a regression. I have run in the past into multiple bugs affecting a bisect. "git checkout a48455c735544fc53696ef7ec84f1da0ae8e7c74" ==> BROKEN "git checkout a48455c735544fc53696ef7ec84f1da0ae8e7c74^" ==> GOOD Previous HEAD position was a48455c7355 comctl32: Remove user32 control copies in comctl32 v5. HEAD is now at 03dd577a243 comctl32/tests: Add RegisterClassNameW() tests. Anyway, the MR https://gitlab.winehq.org/wine/wine/-/merge_requests/9736 was merged and it makes no difference. Chicken Tournament still doesn't starts for me. I tried reverting a48455c735544fc53696ef7ec84f1da0ae8e7c74 but that's not revertable as other changes depend on that. I'm attaching the WINEDEBUG='+seh,+button,+combo,+commctrl,+edit,+listbox,+static' for the three cases. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 --- Comment #4 from Michael Stefaniuc <mstefani@winehq.org> --- Created attachment 79920 --> http://bugs.winehq.org/attachment.cgi?id=79920 BAD: log with EGL OOM fix in wine-11.0-rc1-47-ge5d435e3b80 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 --- Comment #5 from Michael Stefaniuc <mstefani@winehq.org> --- Created attachment 79921 --> http://bugs.winehq.org/attachment.cgi?id=79921 BAD: log with culprit commit wine-10.16-256-ga48455c7355 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 --- Comment #6 from Michael Stefaniuc <mstefani@winehq.org> --- Created attachment 79922 --> http://bugs.winehq.org/attachment.cgi?id=79922 GOOD: log with previous commit wine-10.16-255-g03dd577a243 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Michael Stefaniuc <mstefani@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|ChickenTournament doesn't |ChickenTournament doesn't |starts since the user32 |starts since the user32 |control copies were removed |control copies were removed |from comctl32 v5 |from comctl32 v5 (32bit | |bottle) -- 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.
http://bugs.winehq.org/show_bug.cgi?id=59086 Michael Stefaniuc <mstefani@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |INVALID --- Comment #7 from Michael Stefaniuc <mstefani@winehq.org> --- OK, that was the magic incantation: I tried with a 64bit bottle and current Wine and CT.exe started. Just to cross check I tried with a fresh 32bit bottle and CT.exe starts too. And I no longer see a ton of trace:seh:NtGetContextThread in the logs. So I'm assuming something in the old bottle wasn't updated correctly with the new Wine versions. -- 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 (2)
-
WineHQ Bugzilla -
WineHQ Bugzilla