http://bugs.winehq.org/show_bug.cgi?id=24723
Summary: Lost Horizon crash/page fault during Chapter 2 Product: Wine Version: 1.3.4 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: mblackwell1024@gmail.com
Created an attachment (id=31233) --> (http://bugs.winehq.org/attachment.cgi?id=31233) Console/Debug log
Lost Horizon plays perfectly until Chapter 2. During this mission there is one scene where during dialog it crashes with a page fault.
Attached a full log.
http://bugs.winehq.org/show_bug.cgi?id=24723
Jonathan Strander mblackwell1024@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |http://www.deepsilver.com/e | |n/game-view/view/game/acces | |s16/lost-horizon-pc-1/
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #1 from Jonathan Strander mblackwell1024@gmail.com 2010-11-14 21:24:41 CST --- Still exists on Wine 1.3.7
http://bugs.winehq.org/show_bug.cgi?id=24723
Matěj Týč matej.tyc@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |matej.tyc@gmail.com
--- Comment #2 from Matěj Týč matej.tyc@gmail.com 2010-11-21 11:49:49 CST --- I have the same Wine, but slightly different version of the game (bought over the Internet, so it runs even without noCD crack).
However, I experience the exact behavior. I am offering to donate one copy of the game for development puroposes :-), send me an e-mail if interested.
http://bugs.winehq.org/show_bug.cgi?id=24723
Paul The Tall paulthetall@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |paulthetall@gmail.com
--- Comment #3 from Paul The Tall paulthetall@gmail.com 2011-02-28 06:43:15 CST --- (In reply to comment #2)
I have the same Wine, but slightly different version of the game (bought over the Internet, so it runs even without noCD crack). However, I experience the exact behavior. I am offering to donate one copy of the game for development puroposes :-), send me an e-mail if interested.
It looks like a soundissue. I have the same issue, i got that problem in the monestary when the germans want to kill the leading character. I tried: winetrick dsound to native, makes the sound stotter, still crashes...then tried directmusic. No effect still. tried other wineversions (1.40 and up) same issue. Wineversion below 1.1.25 and a few up crashes already at start...no effect. deleted dsound in libraries. Also no effect.
Try in your case to winetrick sound=alsa however i am looking into it too, but if real developers can look into this too would be nice :)
http://bugs.winehq.org/show_bug.cgi?id=24723
Sander sander@colasiteoflife.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sander@colasiteoflife.com
--- Comment #4 from Sander sander@colasiteoflife.com 2011-06-13 16:24:57 CDT --- I have the same issue with the latest wine 1.3 version from the ubuntu wine ppa.
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #5 from Sander sander@colasiteoflife.com 2011-06-13 16:34:36 CDT --- I have 'fixed' this problem by setting the animation quality and everything to 'low' and shadows to 'off'. I was able to finish chapter 2 and watch the scene.
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #6 from Sander sander@colasiteoflife.com 2011-06-13 17:19:06 CDT --- The game now crashes at chapter 3 while I talk to Hayes..
http://bugs.winehq.org/show_bug.cgi?id=24723
Sander sander@colasiteoflife.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever Confirmed|0 |1
--- Comment #7 from Sander sander@colasiteoflife.com 2011-06-14 06:36:29 CDT --- *** This bug has been confirmed by popular vote. ***
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #8 from Paul The Tall paulthetall@gmail.com 2011-06-14 08:09:09 CDT --- anyone can test the game using sound on emulation (in wincfg on the audio tab?) thanks.
http://bugs.winehq.org/show_bug.cgi?id=24723
dgonsan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dgonsan@gmail.com
--- Comment #9 from dgonsan@gmail.com 2011-07-07 13:18:19 CDT --- Wine 1.3.23 same issue as described in previous posts. I have tested it with pulse, alsa and hardware/emulation and all combinations fail in the same spots, at the end of chapter 2 or when you talk to Hayes in chapter 3.
The only workaround that I have found is disabling completely the sound, in thhat case the game works fine and doesn't hang in the places mentioned above, but obviously playing without sound isn't the best game experience.
http://bugs.winehq.org/show_bug.cgi?id=24723
prawdziwy.adres@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |prawdziwy.adres@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #10 from GyB gyebro69@gmail.com 2011-08-19 00:13:01 CDT --- Created an attachment (id=36046) --> (http://bugs.winehq.org/attachment.cgi?id=36046) console log, Wine-1.3.26
I 'm getting the same problem in Wine-1.3.26: the game always crashes at the exact same point, during one of the conversations at the end of Chapter 2.
Workaround: using native msacm32.dll
Here I'm attaching a new log, the backtrace indicates some heap corruption.
When running the game with WINEDEBUG=warn+heap the following lines are printed out, just before the crash occurs: err:heap:HEAP_ValidateInUseArena Heap 0x2aa5000: in-use arena 0xc9d6020 next block 0xc9e6030 has PREV_FREE flag 1800a warn:heap:HEAP_ValidateInUseArena Heap 0x2aa5000: invalid in-use arena magic 00eefeee for 0xca0e080
Fedora 15 x86 Kernel 2.6.38.8 Alsa 1.0.24 Pulseaudio is not running
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #11 from Paul The Tall paulthetall@gmail.com 2011-08-19 01:18:17 CDT --- With this it doenst crash now? And with what wine version?
http://bugs.winehq.org/show_bug.cgi?id=24723
GyB gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gyebro69@gmail.com
--- Comment #12 from GyB gyebro69@gmail.com 2011-08-19 01:24:19 CDT --- (In reply to comment #11)
With this it doenst crash now? And with what wine version?
Wine-1.3.26. With a native msacm32.dll I could get past the ominous part of the game.
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #13 from Paul The Tall paulthetall@gmail.com 2011-08-19 08:03:04 CDT --- (In reply to comment #12)
(In reply to comment #11)
With this it doenst crash now? And with what wine version?
Wine-1.3.26. With a native I could get past the ominous part of the game.
Confirmed that the game works with wine 1.3.24 a downloaded msacm32.dll put in the system32 folder, and in winecfg i set the msacm32 to native windows.
In wine 1.3.25 or 1.3.26 the game gives no sound at all also not with sound emulated. So in 1.3.24 it works only for the Mac (Core Audio, ALSA doens't work on the Mac)
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #14 from Jonathan Strander mblackwell1024@gmail.com 2011-12-13 18:42:47 CST --- Still crashes for me in the same location using Wine 1.3.34
http://bugs.winehq.org/show_bug.cgi?id=24723
Jörg Höhle hoehle@users.sourceforge.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |msacm32
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #15 from GyB gyebro69@gmail.com 2012-07-08 08:11:47 CDT --- Still present in Wine 1.5.8 Native msacm32 acts as a workaround.
http://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #16 from Jonathan Strander mblackwell1024@gmail.com 2012-07-16 13:30:27 CDT --- Just posting to confirm what GyB said. Must be a missing function?
http://bugs.winehq.org/show_bug.cgi?id=24723
N. de Jonge mail@norbertdejonge.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mail@norbertdejonge.nl
--- Comment #17 from N. de Jonge mail@norbertdejonge.nl 2013-07-11 06:43:04 CDT --- Still a bug in Wine 1.6-rc4. GyB's workaround works.
A bit more information, including a save file that could be used for testing purposes: http://www.norbertdejonge.nl/misc/Lost_Horizon/
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #18 from Austin English austinenglish@gmail.com --- This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.37 or newer) wine? If so, please attach the terminal output in 1.7.37 (see http://wiki.winehq.org/FAQ#get_log).
https://bugs.winehq.org/show_bug.cgi?id=24723
Béla Gyebrószki gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #31233|0 |1 is obsolete| | Attachment #36046|0 |1 is obsolete| |
--- Comment #19 from Béla Gyebrószki gyebro69@gmail.com --- Created attachment 50835 --> https://bugs.winehq.org/attachment.cgi?id=50835 crash log (Wine 1.7.37)
Still present in 1.7.37
Fedora 21 Alsa 1.0.28 Nvidia binary drivers 340.76
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #20 from Bruno Jesus 00cpxxx@gmail.com --- It looks like the demo can't be used to reproduce because it only has parts of the first chapter. Is this still an issue in wine-git? If yes perhaps a +msacm,+relay would help.
Recently a invalid memory write was fixed in msadp32, msacm32 asks different dlls to check if they can play a file so if msadp32 was used in the process it could be related to the crash.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #21 from Béla Gyebrószki gyebro69@gmail.com --- Created attachment 52426 --> https://bugs.winehq.org/attachment.cgi?id=52426 partial +relay,+seh,+tid,+msacm (uncompressed 125M)
Still present in wine-1.7.51-225-g3966aff
Native msacm32.dll works around the crash.
A +relay,+msacm log is huge (1 GB) because the crash occurs near the end of a lengthy conversation. The attached file contains the last 2 million lines.
https://bugs.winehq.org/show_bug.cgi?id=24723
Béla Gyebrószki gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL|http://www.deepsilver.com/e |http://store.steampowered.c |n/game-view/view/game/acces |om/app/40350/ |s16/lost-horizon-pc-1/ |
https://bugs.winehq.org/show_bug.cgi?id=24723
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #22 from Bruno Jesus 00cpxxx@gmail.com --- Created attachment 52813 --> https://bugs.winehq.org/attachment.cgi?id=52813 test patch
This is a shoot in the dark based on the log, right before the crash there are some weird numbers showing that there is possibly not enough space to do the pcm data conversion.
002b:trace:msacm:PCM_StreamConvert (0x26064c, 0x307e8cc) 002b:trace:msacm:PCM_StreamConvert nsrc=9,adsh->cbSrcLength=36 002b:trace:msacm:PCM_StreamConvert ndst=1073741823,adsh->cbDstLength=-4
Maybe by ignoring the conversion we won't write past the output buffer.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #23 from Béla Gyebrószki gyebro69@gmail.com --- (In reply to Bruno Jesus from comment #22)
Created attachment 52813 [details] test patch
This is a shoot in the dark based on the log, right before the crash there are some weird numbers showing that there is possibly not enough space to do the pcm data conversion.
002b:trace:msacm:PCM_StreamConvert (0x26064c, 0x307e8cc) 002b:trace:msacm:PCM_StreamConvert nsrc=9,adsh->cbSrcLength=36 002b:trace:msacm:PCM_StreamConvert ndst=1073741823,adsh->cbDstLength=-4
Maybe by ignoring the conversion we won't write past the output buffer.
The patch fixes the crash, thanks Bruno.
https://bugs.winehq.org/show_bug.cgi?id=24723
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|wine-bugs@winehq.org |00cpxxx@gmail.com
--- Comment #24 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to Béla Gyebrószki from comment #23)
The patch fixes the crash, thanks Bruno.
Nice, thanks! I'm working in a proper patch.
https://bugs.winehq.org/show_bug.cgi?id=24723
super_man@post.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |super_man@post.com
--- Comment #25 from super_man@post.com --- (In reply to Bruno Jesus from comment #24)
(In reply to Béla Gyebrószki from comment #23)
The patch fixes the crash, thanks Bruno.
Nice, thanks! I'm working in a proper patch.
patch from http://source.winehq.org/patches/data/116843 was committed. Care to retest?
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #26 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to super_man from comment #25)
(In reply to Bruno Jesus from comment #24)
(In reply to Béla Gyebrószki from comment #23)
The patch fixes the crash, thanks Bruno.
Nice, thanks! I'm working in a proper patch.
patch from http://source.winehq.org/patches/data/116843 was committed. Care to retest?
That was a list mistake, the patch was wrong. I'm working on this, will take more time.
https://bugs.winehq.org/show_bug.cgi?id=24723
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #52813|0 |1 is obsolete| |
--- Comment #27 from Bruno Jesus 00cpxxx@gmail.com --- Created attachment 52918 --> https://bugs.winehq.org/attachment.cgi?id=52918 patch 2
Please if possible test this patch, it should be a more proper approach. Please grab a +msacm with the patch working or not.
https://bugs.winehq.org/show_bug.cgi?id=24723
Bruno Jesus 00cpxxx@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum@codeweavers.com
--- Comment #28 from Bruno Jesus 00cpxxx@gmail.com --- Andrew, please take a look at the patch from comment 27. Do you think I should move the code into the driver message ACMDM_STREAM_PREPARE? I think so but I didn't find any documentation to backup by sixth sense.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #29 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to Bruno Jesus from comment #28)
Andrew, please take a look at the patch from comment 27. Do you think I should move the code into the driver message ACMDM_STREAM_PREPARE? I think so but I didn't find any documentation to backup by sixth sense.
Actually the driver does not receive the information required to check this because the struct is zeroed, from msacm32/stream.c
padsh->fdwPrepared = 0; padsh->dwPrepared = 0; padsh->pbPreparedSrc = 0; padsh->cbPreparedSrcLength = 0; padsh->pbPreparedDst = 0; padsh->cbPreparedDstLength = 0;
ret = MSACM_Message((HACMDRIVER)was->pDrv, ACMDM_STREAM_PREPARE, (LPARAM)&was->drvInst, (LPARAM)padsh);
But maybe this is wrong in the wine side (?)
https://bugs.winehq.org/show_bug.cgi?id=24723
N. de Jonge mail@norbertdejonge.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC|mail@norbertdejonge.nl |
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #30 from Béla Gyebrószki gyebro69@gmail.com --- Created attachment 52937 --> https://bugs.winehq.org/attachment.cgi?id=52937 +msacm log (patch v2 applied, uncompressed 360 MB)
(In reply to Bruno Jesus from comment #27)
Created attachment 52918 [details] patch 2
Please if possible test this patch, it should be a more proper approach. Please grab a +msacm with the patch working or not.
Patch v2 doesn't work, the game crashes at the same point during conversation. Attached +msacm log with the patch applied.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #31 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to Béla Gyebrószki from comment #30)
Patch v2 doesn't work, the game crashes at the same point during conversation. Attached +msacm log with the patch applied.
Thank you so much as always. Patch v2 fixes some problems but that don't affect this application. Patch v1 seems more correct in this situation, I'm working with the logs to replicate the exact game behavior and create new tests.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #32 from Andrew Eikum aeikum@codeweavers.com --- (In reply to Bruno Jesus from comment #28)
Andrew, please take a look at the patch from comment 27. Do you think I should move the code into the driver message ACMDM_STREAM_PREPARE? I think so but I didn't find any documentation to backup by sixth sense.
Sorry, what's the state of this bug? Is this patch still worth reviewing?
Using native msacm32 is a workaround, so it sounds like the bug is in msacm32 and not in one of the .acm DLLs.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #33 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to Andrew Eikum from comment #32)
Sorry, what's the state of this bug? Is this patch still worth reviewing?
Don't worry, this is not going anywhere during code freeze anyway =) Please do review the patch as it fixes different errors anyway (from my queued tests from wine-patches).
Using native msacm32 is a workaround, so it sounds like the bug is in msacm32 and not in one of the .acm DLLs.
I see, I believe the problem lies in acmStreamConvert and I still believe it is related to the negative size of the output buffer. I will work on creating a test with the exact values from the log to check.
https://bugs.winehq.org/show_bug.cgi?id=24723
--- Comment #34 from Bruno Jesus 00cpxxx@gmail.com --- I have recently worked on this and until now all I was able to do is to make Windows crash the same way Wine does by copying the exact input data from the log =/
Still working...
https://bugs.winehq.org/show_bug.cgi?id=24723
Béla Gyebrószki gyebro69@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Fixed by SHA1| |cadedd73f968fc981e86e020cfe | |00b32ce9de752 Resolution|--- |FIXED
--- Comment #35 from Béla Gyebrószki gyebro69@gmail.com --- This bug was fixed with https://source.winehq.org/git/wine.git/commit/cadedd73f968fc981e86e020cfe00b...
https://bugs.winehq.org/show_bug.cgi?id=24723
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #36 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 2.12.
https://bugs.winehq.org/show_bug.cgi?id=24723
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.0.x
https://bugs.winehq.org/show_bug.cgi?id=24723
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.0.x |---
--- Comment #37 from Michael Stefaniuc mstefani@winehq.org --- Removing the 2.0.x milestone from bugs included in 2.0.3.
-
wine-bugs＠winehq.org