http://bugs.winehq.org/show_bug.cgi?id=20816
Summary: Civilization II crashes on startup when sound is enabled in Wine Product: Wine Version: 1.1.33 Platform: PC OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winmm&mci AssignedTo: wine-bugs@winehq.org ReportedBy: arethusa26@gmail.com CC: eric.pouech@orange.fr
Created an attachment (id=24932) --> (http://bugs.winehq.org/attachment.cgi?id=24932) Civilization II backtrace with +winmm,+mmsys traces
After patching Wine to deal with bug 20806, I noticed Civilization II crashed at a different point with the patch on top of wine-1.1.33-358-gddaf238, which didn't happen when I applied the patch from bug 20806 on top of wine-1.1.30-5-g6ad9518. A regression test indicated:
32c170344ae69d53f0446adf8eb660f2bff0b133 is the first bad commit commit 32c170344ae69d53f0446adf8eb660f2bff0b133 Author: Eric Pouech eric.pouech@orange.fr Date: Thu Oct 22 22:10:25 2009 +0200
winmm: Move the 16bit management of wave out driver type to the new 16=>32 thunking scheme.
:040000 040000 f1d797ca26bfc8fcaf7ea7424fa90232a34052b6 2ae7e2c192aa5dc344404ed597a7d4a60165a636 M dlls
Using Wine prior to this commit or disabling all sound backends in winecfg lets Civilization II start without crashing.
http://bugs.winehq.org/show_bug.cgi?id=20816
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
http://bugs.winehq.org/show_bug.cgi?id=20816
Hanusz Leszek leszek.hanusz@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |leszek.hanusz@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=20816
Eric Pouech eric.pouech@orange.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|wine-bugs@winehq.org |eric.pouech@orange.fr
--- Comment #1 from Eric Pouech eric.pouech@orange.fr 2010-06-19 03:50:33 --- Created an attachment (id=28947) --> (http://bugs.winehq.org/attachment.cgi?id=28947) Force parameters conversion for all callback kinds
Please test with attached patch if this fixes the issue ?
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #2 from Andrew Nguyen arethusa26@gmail.com 2010-06-19 04:35:47 --- I get a core dump. I couldn't trivially get a backtrace with gdb with the core file, so do you want me to attach it compressed?
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #3 from Andrew Nguyen arethusa26@gmail.com 2010-06-19 04:38:54 --- Created an attachment (id=28948) --> (http://bugs.winehq.org/attachment.cgi?id=28948) Civilization II +winmm trace with patch applied
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #4 from Eric Pouech eric.pouech@orange.fr 2010-06-19 04:51:28 --- did you try running the program under winedbg to get the backtrace ?
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #5 from Andrew Nguyen arethusa26@gmail.com 2010-06-19 05:00:47 --- (In reply to comment #4)
did you try running the program under winedbg to get the backtrace ?
Unfortunately, if I try to do this, I get an excessively large amount of first-chance exceptions, and if tell winedbg to ignore first-chance exceptions, then I don't get to capture any backtrace, as civ2.exe just terminates without an unhandled exception.
http://bugs.winehq.org/show_bug.cgi?id=20816
Eric Pouech eric.pouech@orange.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #28947|0 |1 is obsolete| |
--- Comment #6 from Eric Pouech eric.pouech@orange.fr 2010-06-19 05:06:01 --- Created an attachment (id=28949) --> (http://bugs.winehq.org/attachment.cgi?id=28949) Updated patch
please recheck with this patch (I left over a silly mistake)
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #7 from Andrew Nguyen arethusa26@gmail.com 2010-06-19 05:16:22 --- The patch allows me to successfully reach the main menu. I get a crash when trying to start a new game, but I need to check to see if it's related to this issue.
http://bugs.winehq.org/show_bug.cgi?id=20816
--- Comment #8 from Andrew Nguyen arethusa26@gmail.com 2010-06-19 05:59:48 --- The crash with the ALSA backend enabled when trying to start a new game still occurs when I move back to commit 32c170344ae69d53f0446adf8eb660f2bff0b133^ and a0515b66b284bd7012d22d97a4e99fac9ac1c4b1 cherry-picked, so I'm going to assume that's a separate issue.
http://bugs.winehq.org/show_bug.cgi?id=20816
Andrew Nguyen arethusa26@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #9 from Andrew Nguyen arethusa26@gmail.com 2010-06-21 11:25:43 --- A modified version was committed as 6f778a79f4f0e0c229d7dc7917a5fb5195dabd35, marking fixed.
http://bugs.winehq.org/show_bug.cgi?id=20816
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #10 from Alexandre Julliard julliard@winehq.org 2010-06-25 12:41:22 --- Closing bugs fixed in 1.2-rc5.