http://bugs.winehq.org/show_bug.cgi?id=21993
Summary: Lemmings Paintball: Crash when skipping intro video Product: Wine Version: 1.1.40 Platform: x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: ezekiel000@lavabit.com CC: hoehle@users.sourceforge.net
I used to be able to skip the intro video of Lemmings Paintball either by clicking or pressing space bar. But in 1.1.40 it freezes the game so I did a regression test and came up with this: a34831df13a9609566fe3983f5439a7c76890f60 is first bad commit commit a34831df13a9609566fe3983f5439a7c76890f60 Author: Jörg Höhle hoehle@users.sourceforge.net Date: Thu Nov 26 08:46:55 2009 +0100
mciavi: First check inputs, then (perhaps) stop and act.
:040000 040000 7fecce732e9eba748da01693394fddf23eff54ec c809e6183710d218d0dc37e02ce87d968d4648e3 M dlls
http://bugs.winehq.org/show_bug.cgi?id=21993
David Rogers ezekiel000@lavabit.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #1 from Austin English austinenglish@gmail.com 2010-03-09 12:17:57 --- Terminal output?
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #2 from David Rogers ezekiel000@lavabit.com 2010-03-09 12:28:32 --- Created an attachment (id=26715) --> (http://bugs.winehq.org/attachment.cgi?id=26715) Terminal output
Sorry I forgot.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #3 from Jörg Höhle hoehle@users.sourceforge.net 2010-03-10 04:50:16 --- Does using 1.1.40 but reverting solely this patch (git revert) work like it did before?
What happens if you do not click or press the space bar. Does the app proceed to the main menu, i.e. is there a regression only when skipping the video?
Do you have an idea what causes the following (probably unrelated) error on your console? err:iccvid:decode_cinepak CVID: corruption 230400 (QT/AVI) != 0 (CV) Did you skip the intro video because it would not display correctly (that would be a distinct bug) or simply to reach the main menu fast? Can you identify the file that the SW wants played? What does "file xyz.avi" report as format?
Please append a WINEDEBUG=+mci,+mciavi,+driver log.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #4 from David Rogers ezekiel000@lavabit.com 2010-03-10 07:40:41 --- Created an attachment (id=26729) --> (http://bugs.winehq.org/attachment.cgi?id=26729) WINEDEBUG=+mci,+mciavi,+driver without skipping the video
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #5 from David Rogers ezekiel000@lavabit.com 2010-03-10 07:41:06 --- Created an attachment (id=26730) --> (http://bugs.winehq.org/attachment.cgi?id=26730) WINEDEBUG=+mci,+mciavi,+driver with skipping the video
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #6 from David Rogers ezekiel000@lavabit.com 2010-03-10 08:06:24 --- (In reply to comment #3)
Does using 1.1.40 but reverting solely this patch (git revert) work like it did before?
When I tried to revert this patch I got an error: ezekiel@alice:~/Games/Lemmings Paintball/wine-git$ git show a34831df13a9609566fe3983f5439a7c76890f60 | patch -p1 -R patching file dlls/mciavi32/mciavi.c Hunk #3 FAILED at 676. Hunk #4 FAILED at 718. Hunk #5 succeeded at 743 (offset 6 lines). Hunk #6 FAILED at 786. Hunk #7 succeeded at 825 (offset 26 lines). Hunk #8 succeeded at 846 (offset 28 lines). Hunk #9 succeeded at 865 (offset 28 lines). Hunk #10 succeeded at 884 (offset 28 lines). Hunk #11 succeeded at 903 (offset 28 lines). 3 out of 11 hunks FAILED -- saving rejects to file dlls/mciavi32/mciavi.c.rej
I wasn't sure what to do as I'm just following the wiki instructions. But I tried to build it anyway and the error message (see below) still appears but I am able to skip the video without the game freezing.
What happens if you do not click or press the space bar. Does the app proceed to the main menu, i.e. is there a regression only when skipping the video?
The regression is only when skipping the video, if you let it play the video finishes and the game goes into the main menu. Also before the video plays an error box comes up saying MCI ERROR: Unable to set MCI Draw procedure.
Do you have an idea what causes the following (probably unrelated) error on your console? err:iccvid:decode_cinepak CVID: corruption 230400 (QT/AVI) != 0 (CV)
I think this is a problem with the cinepak decoder as it happens on another game that uses cinepak (Starship Titanic) but the videos look fine in Lemmings Paintball (Starship Titanic has loads of problems some videos play others leave a black square on the screen where they should be others play but without sound)
Did you skip the intro video because it would not display correctly (that would be a distinct bug) or simply to reach the main menu fast?
I just skipped it as it's a long intro video with the game credits the video plays fine.
Can you identify the file that the SW wants played? What does "file xyz.avi" report as format?
intro.avi: RIFF (little-endian) data, AVI, 320 x 240, ~15 fps, video: Cinepak, audio: uncompressed PCM (stereo, 11025 Hz)
Please append a WINEDEBUG=+mci,+mciavi,+driver log.
See two attached files. I wasn't sure if you wanted the log from skipping the video or letting it play through.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #7 from David Rogers ezekiel000@lavabit.com 2010-03-10 09:23:45 --- I also noticed that the game also freezes if you select exit from the file menu while the intro video is playing.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #8 from David Rogers ezekiel000@lavabit.com 2010-03-10 09:28:44 --- Sorry to keep adding things but I also noticed that the audio is out of sync with the video, this also happens in totem. Not sure if that is relevant.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #9 from Jörg Höhle hoehle@users.sourceforge.net 2010-03-10 09:42:48 --- Out of sync is irrelevant to the present issue. I've already encountered this with other videos as well. xine performed better.
Also before the video plays an error box comes up saying MCI ERROR: Unable to set MCI Draw procedure.
You did not mention whether this error only comes up since the regression or whether that has always been the case.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #10 from David Rogers ezekiel000@lavabit.com 2010-03-10 09:49:21 --- (In reply to comment #9)
Also before the video plays an error box comes up saying MCI ERROR: Unable to set MCI Draw procedure.
You did not mention whether this error only comes up since the regression or whether that has always been the case.
Sorry this error message comes up since the regression.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #11 from Jörg Höhle hoehle@users.sourceforge.net 2010-03-11 02:44:11 --- Somebody please mark this issue as confirmed. I sent a patch to fix the deadlock to wine-patches today: http://www.winehq.org/pipermail/wine-patches/2010-March/085649.html It's surprising that the regression did not cause ealier bug reports, as any AVIVIDEO closed while playing (not stopped) would deadlock. Probably other apps send the stop command first.
Sorry this error message comes up since the regression.
I didn't look at that yet.
http://bugs.winehq.org/show_bug.cgi?id=21993
Jeff Zaroyko jeffz@jeffz.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch Status|UNCONFIRMED |NEW Ever Confirmed|0 |1
--- Comment #12 from Jeff Zaroyko jeffz@jeffz.name 2010-03-11 03:25:26 --- confirming.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #13 from Jörg Höhle hoehle@users.sourceforge.net 2010-03-15 05:25:16 --- Created an attachment (id=26817) --> (http://bugs.winehq.org/attachment.cgi?id=26817) Patch to avoid MCIERR_UNSUPPORTED in SetVideo + SetAudio stubs
Sorry this error message comes up since the regression.
I believe this is not caused by the commit you bisected rather by its successor (the one about about returning MCIERR_UNSUPPORTED_FUNCTION instead of 0). Please verify that the attached patch fixes this second issue.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #14 from David Rogers ezekiel000@lavabit.com 2010-03-15 15:51:46 --- (In reply to comment #13)
Created an attachment (id=26817)
--> (http://bugs.winehq.org/attachment.cgi?id=26817) [details]
Patch to avoid MCIERR_UNSUPPORTED in SetVideo + SetAudio stubs
Sorry this error message comes up since the regression.
I believe this is not caused by the commit you bisected rather by its successor (the one about about returning MCIERR_UNSUPPORTED_FUNCTION instead of 0). Please verify that the attached patch fixes this second issue.
That patch fixes the second problem with it the error message doesn't pop up any more.
http://bugs.winehq.org/show_bug.cgi?id=21993
--- Comment #15 from Jörg Höhle hoehle@users.sourceforge.net 2010-03-17 12:48:15 --- commit 3f529a6e7068b610b7ab4456eddc835449dc894c fixes the crash. commit 33d1e593aacc3ab1d0b427374d9fc46ba6acfa97 removes the error message.
David, please mark this issue as "resolved" so it will become closed with the next release.
http://bugs.winehq.org/show_bug.cgi?id=21993
David Rogers ezekiel000@lavabit.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #16 from David Rogers ezekiel000@lavabit.com 2010-03-17 13:16:30 --- Done.
http://bugs.winehq.org/show_bug.cgi?id=21993
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #17 from Alexandre Julliard julliard@winehq.org 2010-03-19 14:11:04 --- Closing bugs fixed in 1.1.41.