http://bugs.winehq.org/show_bug.cgi?id=29065
Bug #: 29065 Summary: Phoenix (steam file extractor) hangs when opening Product: Wine Version: 1.3.32 Platform: x86 URL: http://stat1cv01d.com/load/phoenix_1_5_beta_8/1-1-0-13 OS/Version: Linux Status: NEW Keywords: download Severity: normal Priority: P2 Component: user32 AssignedTo: wine-bugs@winehq.org ReportedBy: austinenglish@gmail.com Classification: Unclassified
Created attachment 37450 --> http://bugs.winehq.org/attachment.cgi?id=37450 relay,seh,tid trace
80056a42f6b0e96426751497a8c322de0cdf0602 Phoenix_15beta8.rar
Bernhard had a quick look: <jhgf2> seems to be some bug in the dialog handling code <jhgf2> this thing expects a WM_INITDIALOG message, but wine never sends it
The bug can be reproduced without Steam, just download, extract, and run. The program draws a window, but it's never filled, and the program just hangs.
Terminal output is short: fixme:mountmgr:harddisk_ioctl Unsupported ioctl 74080 (device=7 access=1 func=20 method=0) fixme:mountmgr:harddisk_ioctl Unsupported ioctl 2d1400 (device=2d access=0 func=500 method=0) fixme:mountmgr:harddisk_ioctl Unsupported ioctl 74080 (device=7 access=1 func=20 method=0)
(but unrelated..)
I'll attach logs.
http://bugs.winehq.org/show_bug.cgi?id=29065
--- Comment #1 from Austin English austinenglish@gmail.com 2011-11-11 14:10:43 CST --- Created attachment 37451 --> http://bugs.winehq.org/attachment.cgi?id=37451 +dialog trace
http://bugs.winehq.org/show_bug.cgi?id=29065
jhgf bernhardloos@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardloos@googlemail.com
--- Comment #2 from jhgf bernhardloos@googlemail.com 2011-11-11 18:54:17 CST --- The app tries to create a dialog used as a splash screen in a different thread (0026 in the relay log). The main thread then loops until a static variable is set (all the Sleep calls). The splash screen thread uses DialogBoxParamW to create the dialog, and it is supposed to set the variable in the WM_INITDIALOG handler, but the creation of the dialog fails, so this whole thing hangs.
http://bugs.winehq.org/show_bug.cgi?id=29065
--- Comment #3 from Dmitry Timoshkov dmitry@baikal.ru 2011-11-13 22:46:39 CST --- Created attachment 37489 --> http://bugs.winehq.org/attachment.cgi?id=37489 WS_POPUP takes precedence over WS_POPUP in some cases
Dialog creation fails because of this:
trace:dialog:DIALOG_GetControl32 L"msctls_progress32" L"" 54, 4, 68, 185, 12, d0000001, 00000000, 00000000 trace:win:WIN_CreateWindowEx L"" L"msctls_progress32" ex=00000004 style=d0000001 6,111 278x20 parent=0x100a0 menu=0x36 inst=0x590000 params=(nil) trace:win:dump_window_styles style: WS_POPUP WS_CHILD WS_VISIBLE 00000001 trace:win:dump_window_styles exstyle: WS_EX_NOPARENTNOTIFY trace:win:DestroyWindow (0x100a0) trace:win:WIN_SetWindowLong 0x100a0 0 0 W trace:dialog:EndDialog 0x100a0 0
Attached patch makes it run, but it obviously needs some test cases.
http://bugs.winehq.org/show_bug.cgi?id=29065
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #37489|WS_POPUP takes precedence |WS_CHILD takes precedence description|over WS_POPUP in some cases |over WS_POPUP in some cases
--- Comment #4 from Dmitry Timoshkov dmitry@baikal.ru 2011-11-13 22:48:14 CST --- Comment on attachment 37489 --> http://bugs.winehq.org/attachment.cgi?id=37489 WS_CHILD takes precedence over WS_POPUP in some cases
Correct commit message is "WS_CHILD takes precedence over WS_POPUP in some cases."
http://bugs.winehq.org/show_bug.cgi?id=29065
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
--- Comment #5 from Austin English austinenglish@gmail.com 2011-11-14 15:34:11 CST --- There are still some other minor issues, a virtual desktop seems to help here. I'll file more bugs once this one is fixed.
Thanks for the quick work Dmitry!
http://bugs.winehq.org/show_bug.cgi?id=29065
--- Comment #6 from Dmitry Timoshkov dmitry@baikal.ru 2011-11-15 00:30:01 CST --- Here is a hopefully real fix: http://www.winehq.org/pipermail/wine-patches/2011-November/108813.html
http://bugs.winehq.org/show_bug.cgi?id=29065
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |2429ef905c46aec91465ac187c5 | |f0b528cd35612 Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #7 from Austin English austinenglish@gmail.com 2011-11-15 17:24:13 CST --- http://source.winehq.org/git/wine.git/commitdiff/2429ef905c46aec91465ac187c5... fixes it, thanks Dmitry/Bernhard.
Next bug is bug 29096.
http://bugs.winehq.org/show_bug.cgi?id=29065
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #8 from Alexandre Julliard julliard@winehq.org 2011-11-18 13:03:37 CST --- Closing bugs fixed in 1.3.33.