[Bug 40777] New: The Moment of Silence launcher unusable ( displays only black)
https://bugs.winehq.org/show_bug.cgi?id=40777 Bug ID: 40777 Summary: The Moment of Silence launcher unusable (displays only black) Product: Wine Version: 1.9.12 Hardware: x86 URL: http://www.fileplanet.com/147177/140000/fileinfo/The-M oment-of-Silence-Demo-[English] OS: Linux Status: NEW Keywords: download Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs(a)winehq.org Reporter: gyebro69(a)gmail.com Distribution: --- moslaunch.exe must be the configuration tool/launcher for the game Moment of Silence, but it never worked for me: music begins to play when I start it and a black rectangle appears in the middle of the screen,but nothing can be seen or done (except pressing Alt-F4 to close the launcher). Seems to be a long-standing problem as Appdb contains test results with Wine 1.x describing the same problem. There is no terminal output at all. Can be reproduced with the demo: the launcher will start automatically when installation of the demo is finished. mos_english_demo.exe (295M) sha1: 4e29a69d14e66fce9de4bd74cdb4786a043217ba Fedora 23 Nvidia binary drivers 364.19 (the same problem with nouveau) XFCE 4.12 (compositor disabled) -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 fjfrackiewicz(a)gmail.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fjfrackiewicz(a)gmail.com --- Comment #1 from fjfrackiewicz(a)gmail.com --- Same thing happens on Antergos 4.6.2-1 with the latest Nvidia drivers (364.19). Wine 1.9.12 32-bit prefix in Windows 7 mode. I also get no terminal output and using WINEDEBUG=+all does nothing. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #2 from fjfrackiewicz(a)gmail.com --- Still an issue in Wine 1.9.13. No terminal output because none is generated. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #3 from fjfrackiewicz(a)gmail.com --- Issue is still present in Wine 1.9.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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #4 from fjfrackiewicz(a)gmail.com --- Still an issue with Wine-1.9.23-141-g1e78138 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Roland Häder <roland(a)mxchange.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |roland(a)mxchange.org --- Comment #5 from Roland Häder <roland(a)mxchange.org> --- Still an issue with WINE 4.0.1 (stable), updating to 4.0.2 now but I guess it won't work. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Paul Gofman <gofmanp(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gofmanp(a)gmail.com --- Comment #6 from Paul Gofman <gofmanp(a)gmail.com> --- Created attachment 65457 --> https://bugs.winehq.org/attachment.cgi?id=65457 Fix I've tested the demo. The launcher is getting confused by zero window coordinates passed to WM_NCCALCSIZE when SetWindowPos() is called from SetWindowRgn(). Zero coordinate are passed to WM_WINDOWPOSCHANGING in WINPOS structure both under Windows and Wine, but WM_NCCALCSIZE is different. I am attaching the patch which is fixing it. While that fixes the launcher, the demo still does not start. Choosing 'Start' runs some DRM config / launcher, when selecting 'Ok' there it registers some services and then fails: 003f:fixme:ntoskrnl:PsSetCreateThreadNotifyRoutine stub: 003405C0 wine: Unhandled page fault on read access to 0xffffffff at address 00359C07 (thread 0045), starting debugger... 0034:err:service:process_send_command service protocol error - failed to read pipe r = 0 count = 0! 0032:err:module:LdrInitializeThunk "MOSSFL.DLL" failed to initialize, aborting 0032:err:module:LdrInitializeThunk Initializing dlls for L"C:\\mos\\Mos.exe" failed, status c0000142 Then those services continued to crash during each "boot" of Wine prefix until I removed them manually (while it is really better to delete such Wine prefix). The demo itself didn't work for me on Windows 7 either. There was no crash upon attempt to start the game itself, but it silently closes. Maybe it needs to be run as administrator, but I did not try that. So if this demo really works on Windows somehow (or a full game does) but does not work under Wine these further issues deserve a separate bug report. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #7 from Roland Häder <roland(a)mxchange.org> --- (In reply to Paul Gofman from comment #6)
Created attachment 65457 [details] Fix
I've tested the demo.
The launcher is getting confused by zero window coordinates passed to WM_NCCALCSIZE when SetWindowPos() is called from SetWindowRgn(). Zero coordinate are passed to WM_WINDOWPOSCHANGING in WINPOS structure both under Windows and Wine, but WM_NCCALCSIZE is different.
I am attaching the patch which is fixing it.
Thank you for putting efforts into it, very kind. :-) I hope the patch matches quality minimum so it will make it into WINE repository. I will test (applied flawless) it for you with the retail version. Still the DRM might deserve an other "fix". ;-) -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #8 from Béla Gyebrószki <gyebro69(a)gmail.com> --- The patch fixes the problem in 'The Moment of Silence' as well as in 'Overclocked - A History of Violence' (which uses a similar looking launcher). Tested with the GOG.com version. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #9 from Roland Häder <roland(a)mxchange.org> --- In dlls/user32/tests/msg.c you might want to add an extra line behind the `break;` which is directly above a `case` statement. I saw that many times already in WINE code. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #10 from Roland Häder <roland(a)mxchange.org> --- I think it is now a good time to add `patch` to the tags? -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #11 from Roland Häder <roland(a)mxchange.org> --- (In reply to Roland Häder from comment #10)
I think it is now a good time to add `patch` to the tags?
Er, they are private. Better keywords? Or let the maintainers do it. I guess that is best choice. Because this ticket includes a seems-like working fix. I still have to test it here. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #12 from Paul Gofman <gofmanp(a)gmail.com> --- Should be fixed by 2ac268436b535ed35c82aa2d5c2518e5dfb3cd12. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Béla Gyebrószki <gyebro69(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |2ac268436b535ed35c82aa2d5c2 | |518e5dfb3cd12 Component|-unknown |user32 --- Comment #13 from Béla Gyebrószki <gyebro69(a)gmail.com> --- Works in wine-4.18-54-g98c2c9a9c9, thank you. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 --- Comment #14 from Roland Häder <roland(a)mxchange.org> --- Good work everybody. :-) -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #15 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 4.19. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |4.0.x -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- URL|http://www.fileplanet.com/1 |https://web.archive.org/web |47177/140000/fileinfo/The-M |/20200401190414/http://down |oment-of-Silence-Demo-[Engl |load.fileplanet.com/ftp1/10 |ish] |2004/mos_english_demo.exe?s | |t=xedRChOM0ThCSS1jBduprw&e= | |1585778638 CC| |focht(a)gmx.net -- 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.
https://bugs.winehq.org/show_bug.cgi?id=40777 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.0.x |--- --- Comment #16 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 4.0.x milestone from bug fixes included in 4.0.4. -- 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)
-
wine-bugs@winehq.org -
WineHQ Bugzilla