https://bugs.winehq.org/show_bug.cgi?id=42768
Bug ID: 42768 Summary: Ys: The Ark of Napishtim (Ys 6) silently crashes playing in-game videos Product: Wine Version: 2.5 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: devenum Assignee: wine-bugs@winehq.org Reporter: kq3thih@mailnesia.com Regression SHA1: ff0a25cbc74f358884b2750a75c708a9ad839e04 Distribution: Debian
Created attachment 57778 --> https://bugs.winehq.org/attachment.cgi?id=57778 +tid,+devenum,+seh log
Ys: The Ark of Napishtim (Ys 6) silently exits, no backtrace or additional console ouput, when playing the in-game videos with Wine 2.5 (2.4 works). Clean 32-bit prefix, winegstreamer set to disabled and winetricks amstream quartz for working video playback. Start a new game and once reaching the opening video you hit the crash.
Bisecting returns:
--- snip --- ff0a25cbc74f358884b2750a75c708a9ad839e04 is the first bad commit commit ff0a25cbc74f358884b2750a75c708a9ad839e04 Author: Jactry Zeng jzeng@codeweavers.com Date: Mon Mar 27 23:41:11 2017 +0800
devenum: Use RegQueryValueExW instead of RegQueryValueW.
Signed-off-by: Jactry Zeng jzeng@codeweavers.com Signed-off-by: Andrew Eikum aeikum@codeweavers.com Signed-off-by: Alexandre Julliard julliard@winehq.org
:040000 040000 e74122e6a5c4d5dd5a0334c7c4c0485a679262f1 3206743391112c31ca627312c67eb8fbfa0a0ca9 M dlls --- snip ---
Reverting the commit on wine-2.5 stops the crashing and videos play normally.
https://bugs.winehq.org/show_bug.cgi?id=42768
Aexander kq3thih@mailnesia.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC| |jzeng@codeweavers.com
https://bugs.winehq.org/show_bug.cgi?id=42768
Aexander kq3thih@mailnesia.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum@codeweavers.com
https://bugs.winehq.org/show_bug.cgi?id=42768
Jactry Zeng jactry92@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jactry92@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #1 from Jactry Zeng jactry92@gmail.com --- Hi Aexander, (In reply to Aexander from comment #0)
Created attachment 57778 [details] +tid,+devenum,+seh log
Ys: The Ark of Napishtim (Ys 6) silently exits, no backtrace or additional console ouput, when playing the in-game videos with Wine 2.5 (2.4 works). Clean 32-bit prefix, winegstreamer set to disabled and winetricks amstream quartz for working video playback. Start a new game and once reaching the opening video you hit the crash.
Thanks for testing!
Which version of Ys VI did you test with? I tested it with one from Steam and it worked for me. How about using builtin amstream and quartz?
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #2 from Aexander kq3thih@mailnesia.com --- Created attachment 57782 --> https://bugs.winehq.org/attachment.cgi?id=57782 builtin amstream quartz
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #3 from Aexander kq3thih@mailnesia.com --- (In reply to Jactry Zeng from comment #1)
Which version of Ys VI did you test with? I tested it with one from Steam and it worked for me. How about using builtin amstream and quartz?
GOG version, standalone installer. Also, I was using the ys6_win_dx9.exe, but checking the DX10? EXE, ys6_win.exe, crashes the same way for me.
Using builtin amstream and quartz the videos are silently skipped, but the game proceeds fine otherwise.
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #4 from Aexander kq3thih@mailnesia.com --- Created attachment 57807 --> https://bugs.winehq.org/attachment.cgi?id=57807 mediainfo opening.avi
(In reply to Jactry Zeng from comment #1)
I tested it with one from Steam and it worked for me.
Was this with WINEDLLOVERRIDES=winegstreamer=d and a clean prefix? I don't think vanilla Wine with builtin amstream & quartz should work with this type of video when gstreamer is disabled. I could be wrong about that though.
Unless the Steam build has the videos encoded differently? Attached output of mediainfo for the opening video for the GOG build.
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #5 from Aexander kq3thih@mailnesia.com --- Created attachment 57808 --> https://bugs.winehq.org/attachment.cgi?id=57808 backtrace
Backtrace attached.
Also, native devenum works around the crash and videos play normally.
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #6 from Jactry Zeng jactry92@gmail.com --- Hi, In reply to Aexander from comment #4)
Created attachment 57807 [details] mediainfo opening.avi
(In reply to Jactry Zeng from comment #1)
I tested it with one from Steam and it worked for me.
Was this with WINEDLLOVERRIDES=winegstreamer=d and a clean prefix? I don't think vanilla Wine with builtin amstream & quartz should work with this type of video when gstreamer is disabled. I could be wrong about that though.
Unless the Steam build has the videos encoded differently? Attached output of mediainfo for the opening video for the GOG build.
I can reproduce the 'video silently skipped' bug with Steam too, with builtin quartz and amstream in a clean prefix. But I think this will be considered as a bug of quartz/amstream. And we need to fix quartz/amstream to let it work.
Could you file another bug for that 'video skipped' bug?
Thanks for your work!
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #7 from Aexander kq3thih@mailnesia.com --- Created attachment 59156 --> https://bugs.winehq.org/attachment.cgi?id=59156 devenum log with reverted commit - 2.14
Finally took another look at this. Attached is a devenum log with commit ff0a25cbc74f358884b2750a75c708a9ad839e04 reverted on 2.14, still an issue in 2.16 btw. Game appears to use a different code path when RegQueryValueExW is available (aside from the initial logo video)? When available it's passing the string "GStreamer splitter filter" to DEVENUM_IParseDisplayName_ParseDisplayName: trace:devenum:DEVENUM_IParseDisplayName_ParseDisplayName (0x4e4ec80, L"GStreamer splitter filter", 0x33f07c, 0x6683108) The crash happens as there is no '{' character in that string (no '}' would be the next crash). According to the comments for that function it's expecting strings in the format of "@device:sw:{CLSID1}<filter name or CLSID>".
(In reply to Jactry Zeng from comment #6)
Could you file another bug for that 'video skipped' bug?
Someone already did: bug 43513.
https://bugs.winehq.org/show_bug.cgi?id=42768
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12@gmail.com
--- Comment #8 from Zebediah Figura z.figura12@gmail.com --- Is this still an issue after b0300ce56483fb2f2a03de8aa216d41b0bfd034e?
https://bugs.winehq.org/show_bug.cgi?id=42768
--- Comment #9 from Aexander kq3thih@mailnesia.com --- (In reply to Zebediah Figura from comment #8)
Is this still an issue after b0300ce56483fb2f2a03de8aa216d41b0bfd034e?
No, this working again with builtin devenum, wine-3.3-88-g813ab925ab. Thanks Zebediah.
https://bugs.winehq.org/show_bug.cgi?id=42768
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |b0300ce56483fb2f2a03de8aa21 | |6d41b0bfd034e Status|UNCONFIRMED |RESOLVED
--- Comment #10 from Zebediah Figura z.figura12@gmail.com --- Resolving fixed then.
https://bugs.winehq.org/show_bug.cgi?id=42768
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #11 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.4.
https://bugs.winehq.org/show_bug.cgi?id=42768
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=42768
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #12 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bugs included in 3.0.2.
https://bugs.winehq.org/show_bug.cgi?id=42768
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|devenum |quartz