[Bug 36375] New: eRacer demo doesn't start, fails to load audio files

http://bugs.winehq.org/show_bug.cgi?id=36375

Bug ID: 36375
           Summary: eRacer demo doesn't start, fails to load audio files
           Product: Wine
           Version: 1.7.18
          Hardware: x86
               URL: http://www.gamershell.com/download_5118.shtml
                OS: Linux
            Status: NEW
          Keywords: download, regression
          Severity: normal
          Priority: P2
         Component: winmm&mci
          Assignee: [e-mail]
          Reporter: [e-mail]
            Blocks: 29598
   Regression SHA1: 7a222508679328875b795392f2b90a6f2b313016

Created attachment 48435
  --> http://bugs.winehq.org/attachment.cgi?id=48435
audio debug log (uncompressed 1.9 MB)

To reproduce the problem first you need to rename or remove the intro.avi file
because it causes another crash when starting the demo, but it is unrelated to
the current problem. The file is located in ../eRacer/Gui/Avi directory.

When you start the demo with eracer.exe an error messages is shown, the game
fails to load one of the game audio files (GUI_MUSIC.wav).

This didn't happen before

7a222508679328875b795392f2b90a6f2b313016 is the first bad commit
commit 7a222508679328875b795392f2b90a6f2b313016
Author: Akihiro Sagawa 
Date:   Thu Apr 4 22:28:14 2013 +0900

winmm: Inherit the seek position when using standard file handle.

:040000 040000 d0bf0f390d708af61be7e7c98153516bb00934be
8f1d2c8170972509af5c4a04804af0e3ab2d8082 M    dlls

Reverting the commit on 1.7.18 lets the game get further and it hits another
bug (regression). Plain terminal output doesn't show anything audio related,
here I'm attaching a
+tid,+mmdevapi,+winmm,+driver,+msacm,+midi,+dsound,+dsound3d,+dmusic,+mci,+alsa
debug log.

Fedora 20
Kernel 3.14.2-200.fc20.i686+PAE
Alsa 1.0.27 (pulseaudio is not running)
wine-1.7.18-22-ge399729
Audio device: NVIDIA Corporation MCP61 High Definition Audio (rev a2)

[Bug 36375] eRacer demo doesn't start, fails to load audio files

http://bugs.winehq.org/show_bug.cgi?id=36375

Béla Gyebrószki  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |36376

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

Matteo Bruni  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sagawa.aki+winebugs@gmail.c
                   |                            |om

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

--- Comment #1 from Sagawa  ---
Hello, I can confirm the issue.
I'm inspecting more.

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

--- Comment #2 from Sagawa  ---
Created attachment 49465
  --> https://bugs.winehq.org/attachment.cgi?id=49465
proposed patch

Hi,  Béla
Could you try out this patch?

Actually, this issue isn't my fault (and regression). Even before the commit,
eRacer shows the error message when specifying WINEDEBUG=mmio.

I find out the root cause that wine sends MMIOM_SEEK message while opening a
file. However, in native, MMIOM_SEEK isn't sent at all. Thus, I removed
MMIOM_SEEK usages in mmioOpen() when custom procedure is used. Now, it works on
my PC.

If it works in your environment, too, I'll send the patch to wine-patches.
Thanks in advance.

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

--- Comment #3 from Béla Gyebrószki  ---
(In reply to Sagawa from comment #2)
> Created attachment 49465 [details]
> proposed patch
>
> Hi,  Béla
> Could you try out this patch?
>
> Actually, this issue isn't my fault (and regression). Even before the
> commit, eRacer shows the error message when specifying WINEDEBUG=mmio.
>
> I find out the root cause that wine sends MMIOM_SEEK message while opening a
> file. However, in native, MMIOM_SEEK isn't sent at all. Thus, I removed
> MMIOM_SEEK usages in mmioOpen() when custom procedure is used. Now, it works
> on my PC.
>
> If it works in your environment, too, I'll send the patch to wine-patches.
> Thanks in advance.

The patch works here with eRacer, thank you.

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

--- Comment #4 from Sagawa  ---
Thanks for testing.
I've sent the patch to wine-patches:
https://source.winehq.org/patches/data/106316

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

Béla Gyebrószki  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |e2779a4c9aec515e0e478cb1d4c
                   |                            |9fbe5a87d2367
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Béla Gyebrószki  ---
Fixed by e2779a4c9aec515e0e478cb1d4c9fbe5a87d2367

[Bug 36375] eRacer demo doesn't start, fails to load audio files

https://bugs.winehq.org/show_bug.cgi?id=36375

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.26.