http://bugs.winehq.org/show_bug.cgi?id=20709
Summary: [Regression] 16-bit application page faults on startup Product: Wine Version: 1.1.33 Platform: PC OS/Version: Linux Status: UNCONFIRMED Severity: major Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: perchrh-wine@pvv.org
Created an attachment (id=24739) --> (http://bugs.winehq.org/attachment.cgi?id=24739) The console output when executing 'wine ./setup.exe'
This application works in Windows and worked in Wine 1.1.29.
The logs from the crash are attached. They start like this:
wine: Unhandled page fault on read access to 0xffffffff at address 0x1727:0x00000dbb (thread 001e), starting debugger... Unhandled exception: page fault on read access to 0xffffffff in 16-bit code (1727:0dbb). In 16 bit mode.
The application in question is the game "Mordor - the depths of Dejenol", http://appdb.winehq.org/objectManager.php?sClass=version&iId=8673&iT... Although it did not fully work with previous Wine versions, it was more functional than now (wine 1.1.33).
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #1 from Per Christian Henden perchrh-wine@pvv.org 2009-11-14 13:59:10 --- To reproduce, download the two zip files linked in the applications AppDB entry (link in my post). Unzip them in a new directory, enter the directory and execute 'wine ./SETUP.exe'
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #2 from Wylda wylda@volny.cz 2009-11-14 16:02:06 --- Created an attachment (id=24741) --> (http://bugs.winehq.org/attachment.cgi?id=24741) Proper (i.e. with debuging symbols) console crash log from wine-1.1.33
Crash is caused by Alexandre Julliard's commit...
1. Confirming, please consider UNCONFIRMED->NEW and KEYWORDS: +REGRESSION, +DOWNLOAD, +INSTALLER
2. Did a regression test between 1.1.30 and 1.1.31:
86c6021c71a563af88c1a8cc23ea489e92c8ca70 is first bad commit commit 86c6021c71a563af88c1a8cc23ea489e92c8ca70 Author: Alexandre Julliard julliard@winehq.org Date: Thu Oct 8 19:11:58 2009 +0200
kernel32: Reimplement GetPrivateProfileString16 on top of 32-bit functions and move it to file16.c.
:040000 040000 8c8e93e5bab3d7db358a745922f4ee27f99797e6 cd4b25e37089d23c1fbe1df0e5ee1a481d2a9a8f M dlls
3. No other bug report suffers from this commit.
4. Revert of this patch on top of wine-1.1.31 makes that problem goes away.
5. Adding author of this patch to CC.
--private keyword: bisected
Download: http://decklinsdomain.net/DemDev/archives/Mordor1/MORDOR1A.ZIP http://decklinsdomain.net/DemDev/archives/Mordor1/MORDOR1B.ZIP
http://bugs.winehq.org/show_bug.cgi?id=20709
Wylda wylda@volny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |julliard@winehq.org, | |wylda@volny.cz
http://bugs.winehq.org/show_bug.cgi?id=20709
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression, win16 Summary|[Regression] 16-bit |Mordor - the depths of |application page faults on |Dejenol installer crashes |startup | Severity|major |normal
http://bugs.winehq.org/show_bug.cgi?id=20709
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #3 from Dmitry Timoshkov dmitry@codeweavers.com 2009-11-16 12:42:00 --- Should be fixed in current git.
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #4 from Wylda wylda@volny.cz 2009-11-16 14:33:13 --- Created an attachment (id=24793) --> (http://bugs.winehq.org/attachment.cgi?id=24793) Crash log from wine-1.1.33-88-gd51b4e1
(In reply to comment #3)
Should be fixed in current git.
Hi, Dmitry. Wait wait wait man... :)
Current git (wine-1.1.33-88-gd51b4e1) still crashes, still the winevdm.exe, but this time in different place i guess.
Fresh crash log attached, i.e. not RESOLVED and not FIXED.
http://bugs.winehq.org/show_bug.cgi?id=20709
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |UNCONFIRMED Resolution|FIXED |
--- Comment #5 from Vitaliy Margolen vitaliy@kievinfo.com 2009-11-16 23:04:52 --- Reopening then.
http://bugs.winehq.org/show_bug.cgi?id=20709
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #6 from Dmitry Timoshkov dmitry@codeweavers.com 2009-11-17 01:48:43 --- That's a separate problem, please open another bug report for it. If it's a regression then do the regeression test. This regression *is* fixed.
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #7 from Wylda wylda@volny.cz 2009-11-17 02:09:48 --- (In reply to comment #6)
That's a separate problem, please open another bug report for it. If it's a regression then do the regeression test. This regression *is* fixed.
OK, but i need an advise how to do it...
First bug and second also is probably in between 1.1.30 and 1.1.31, but the patch for first bug is behind 1.1.33. How do i find the second if i cant cross the first bug? I hope you understand me.
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #8 from Dmitry Timoshkov dmitry@codeweavers.com 2009-11-17 02:17:59 --- (In reply to comment #7)
First bug and second also is probably in between 1.1.30 and 1.1.31, but the patch for first bug is behind 1.1.33. How do i find the second if i cant cross the first bug? I hope you understand me.
This regression should be fixed by http://source.winehq.org/git/wine.git/?a=commitdiff;h=935e06c5c37e12405364e2...
So just apply this patch when performing the regression testing. Be prepared that the patch won't apply or will need manual adjustment.
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #9 from Wylda wylda@volny.cz 2009-11-17 02:23:26 --- (In reply to comment #8)
This regression should be fixed by
Hmmm... before firmly "*is*", now careful "should" ;)
Be prepared that the patch won't apply or will need manual adjustment.
Great. This is what i was afraid of. Nu wot, straight in messing my git tree :-))
http://bugs.winehq.org/show_bug.cgi?id=20709
--- Comment #10 from Wylda wylda@volny.cz 2009-11-17 09:08:43 --- (In reply to comment #8)
This regression should be fixed by http://source.winehq.org/git/wine.git/?a=commitdiff;h=935e06c5c37e12405364e2...
Dmitry, you were absolutely right. That patch in current git (wine-1.1.33-88-gd51b4e1) fixed it, but unfortunately it was followed by another commit which broke it again.
So saga continues at bug 20737.
http://bugs.winehq.org/show_bug.cgi?id=20709
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org 2009-12-04 12:16:31 --- Closing bugs fixed in 1.1.34.
http://bugs.winehq.org/show_bug.cgi?id=20709
Wylda wylda@volny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |935e06c5c37e12405364e2ba65a | |f0eedf44d4f85 Regression SHA1| |86c6021c71a563af88c1a8cc23e | |a489e92c8ca70