http://bugs.winehq.org/show_bug.cgi?id=13250
Summary: majesty exits due to page fault Product: Wine Version: 1.0-rc1 Platform: Other OS/Version: other Status: UNCONFIRMED Severity: enhancement Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: jcollins@asgardsrealm.net
Under previous versions (0.9.58 on Debian etch confirmed) Majesty would run without major issue. The videos would not play and needed to be skipped, but the game was playable.
However, with 1.0-rc1 and 0.9.59 on Ubuntu Hardy, the application does not run and exits with something similar to the following each time:
wine: Unhandled page fault on write access to 0x417149cb at address 0x5e9690 (thread 0021), starting debugger... Unhandled exception: page fault on write access to 0x417149cb in 32-bit code (0x005e9690).
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #1 from Jamin Collins jcollins@asgardsrealm.net 2008-05-15 17:03:07 --- Created an attachment (id=13087) --> (http://bugs.winehq.org/attachment.cgi?id=13087) console dump
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #2 from Juan Lang juan_lang@yahoo.com 2008-05-15 17:56:04 --- Please do a regression test: http://wiki.winehq.org/RegressionTesting
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #3 from Jamin Collins jcollins@asgardsrealm.net 2008-05-15 18:51:16 --- Which system would you like the regression testing done on? Hardy doesn't have a version of wine packaged prior to 0.9.59 (which doesn't work) and Debian doesn't appear to have a package post 0.9.58 (which does work).
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #4 from Jamin Collins jcollins@asgardsrealm.net 2008-05-15 22:10:32 --- Grabbed the 0.9.58 source to confirm it worked properly on Ubuntu Hardy, it did.
Performed the regression testing requested. Completing it was somewhat problematic as quite a few of the bisect targets would not compile due to a missing SAX2.h. However, I was finally able to pin it down.
The problematic commit is:
7a7ef6a64c5d69376c86303e7ce6bfc74ea48cfb is first bad commit commit 7a7ef6a64c5d69376c86303e7ce6bfc74ea48cfb Author: Alexandre Julliard julliard@winehq.org Date: Wed Mar 26 17:08:38 2008 +0100
kernel32: Moved the allocation of the process stack to ntdll.
:040000 040000 092913e4e27d91df2086d8c4744a2d4dc047e9f6 aee2ed48b60a4d94a2d1695deaa39206d4dd2c14 M dlls
http://bugs.winehq.org/show_bug.cgi?id=13250
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |julliard@winehq.org Severity|enhancement |critical Status|UNCONFIRMED |NEW Component|-unknown |ntdll Ever Confirmed|0 |1
--- Comment #5 from Vitaliy Margolen vitaliy@kievinfo.com 2008-05-17 01:07:12 --- Confirming - old SafeDisk is broken which used to work for a long time. Looks like the injected code is not being run at all - the code @ the entry point is garbage.
If I recall correctly native allocates stack in the parent process before the actual process started. Moving it into LdrInitializeThunk IMHO is wrong.
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #6 from Alexandre Julliard julliard@winehq.org 2008-05-19 09:13:36 --- Created an attachment (id=13179) --> (http://bugs.winehq.org/attachment.cgi?id=13179) Move signal unmasking
Does this help?
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #7 from Jamin Collins jcollins@asgardsrealm.net 2008-05-19 17:04:16 --- Yes, that appears to correct the problem, at least for Majesty.
http://bugs.winehq.org/show_bug.cgi?id=13250
--- Comment #8 from Vitaliy Margolen vitaliy@kievinfo.com 2008-05-19 22:03:24 --- Fixes here too. Thank you.
http://bugs.winehq.org/show_bug.cgi?id=13250
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #9 from Alexandre Julliard julliard@winehq.org 2008-05-20 09:06:57 --- Fix is committed, thanks for testing it.
http://bugs.winehq.org/show_bug.cgi?id=13250
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #10 from Alexandre Julliard julliard@winehq.org 2008-05-23 10:36:29 --- Closing bugs fixed in 1.0-rc2.