https://bugs.winehq.org/show_bug.cgi?id=56367
Bug ID: 56367 Summary: Tomb Raider 3 GOG crashes at start Product: Wine Version: 9.3 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: d3d Assignee: wine-bugs@winehq.org Reporter: dkk089@gmail.com Distribution: ---
Created attachment 76108 --> https://bugs.winehq.org/attachment.cgi?id=76108 Tomb3.exe backtrace with Wine 9.3
Tomb Raider 3 GOG crashes in Wine 9.3 with the attached backtrace.
Last known working version in AppDB is Wine 4.2. Working upwards from that identified Wine 5.7 as the last working release, and bisecting between 5.7 and 5.8 identified commit 100ca518055c0e9edb68aa370f15a2768cf2c74f as the cause.
Reverting this commit from current master (1b32ac45f821ee1fe06a3dc4f903a81a190216c7) makes the game work normally. Tested with Windows version set to 98, WINEARCH is win32.
This is also reproducible with the demo version available at https://archive.org/details/TombRaiderIiiAdventuresOfLaraCroftDemo .
https://bugs.winehq.org/show_bug.cgi?id=56367
Daniel Kamil Kozar dkk089@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |100ca518055c0e9edb68aa370f1 | |5a2768cf2c74f
https://bugs.winehq.org/show_bug.cgi?id=56367
Daniel Kamil Kozar dkk089@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.winehq.org/sho | |w_bug.cgi?id=39534
https://bugs.winehq.org/show_bug.cgi?id=56367
Daniel Kamil Kozar dkk089@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://archive.org/details | |/TombRaiderIiiAdventuresOfL | |araCroftDemo Keywords| |download, regression
https://bugs.winehq.org/show_bug.cgi?id=56367
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 Status|UNCONFIRMED |NEW
--- Comment #1 from Andrew Nguyen arethusa26@gmail.com --- I can confirm the Tomb Raider 3 demo crashes on wine-9.4.
The read-only protection on the IDirectDraw vtable concealed a buffer overrun that becomes apparent when the vtable was made writable. The overrun occurs when the game modifies during IDirect3D2::EnumDevices enumeration the contents of the buffer holding the device description string for the DirectDraw reference device. The buffer happens to be located just before the IDirectDraw vtable:
Thread 1 "01c0" hit Hardware watchpoint 1: ddraw1_vtbl.Compact
Old value = (HRESULT (*)(IDirectDraw *)) 0x78267b60 <ddraw1_Compact> New value = (HRESULT (*)(IDirectDraw *)) 0x6e6f69
Wine-gdb> print ddraw1_vtbl $2 = {QueryInterface = 0x72614320, AddRef = 0x6d452064, Release = 0x74616c75, Compact = 0x6e6f69, CreateClipper = 0x7826d400 <ddraw1_CreateClipper>, CreatePalette = 0x78262ce0 <ddraw1_CreatePalette>, ... Wine-gdb> print (char *)&ddraw1_vtbl $3 = 0x7829c080 <ddraw1_vtbl> " Card Emulation" Wine-gdb> info symbol (char *)&ddraw1_vtbl - 1 reference_description + 23 in section .data of /home/arethusa/wine32/dlls/ddraw/i386-windows/ddraw.dll Wine-gdb> print (char *)&ddraw1_vtbl - 24 $4 = 0x7829c068 <reference_description> "Core Design MMX Hardware Card Emulation"
Reserving some extra space in the reference_description array declared inside the d3d3_EnumDevices implementation function avoids the buffer overrun and prevents the game from crashing.
I also notice that Windows 8 and newer versions implement an additional integrity protection of the IDirectDraw vtable. The function pointers in the vtable continue to be mutable, but when all IDirectDraw instances are destroyed and a new instance is created again, the original vtable function pointers are restored to undo any modifications.
This remediation logic is not necessary to resolve the crash in the Tomb Raider 3 demo but it would also have been an effective solution, since the game releases all DirectDraw instances it used for device enumeration before it creates a new one for the game launch.
https://bugs.winehq.org/show_bug.cgi?id=56367
--- Comment #2 from Andrew Nguyen arethusa26@gmail.com --- I created a merge request: https://gitlab.winehq.org/wine/wine/-/merge_requests/5272
https://bugs.winehq.org/show_bug.cgi?id=56367
--- Comment #3 from Daniel Kamil Kozar dkk089@gmail.com --- Fantastic job Andrew, thank you very much.
I can confirm that your patch applied to current master (4860fbe7) resolves this issue for the GOG version as well and the game runs just fine.
https://bugs.winehq.org/show_bug.cgi?id=56367
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1| |2c8d4f5abd264f68565bd393bf7 | |c9355ecfd762a Resolution|--- |FIXED
--- Comment #4 from Andrew Nguyen arethusa26@gmail.com --- Fixed by https://source.winehq.org/git/wine.git/commitdiff/2c8d4f5abd264f68565bd393bf...
https://bugs.winehq.org/show_bug.cgi?id=56367
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #5 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 9.5.
https://bugs.winehq.org/show_bug.cgi?id=56367
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |9.0.x
http://bugs.winehq.org/show_bug.cgi?id=56367
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|9.0.x |---
--- Comment #6 from Michael Stefaniuc mstefani@winehq.org --- Removing the 9.0.x milestone from bug fixes included in 9.0.1.
-
WineHQ Bugzilla