[Bug 20848] New: Pumuckl: regained sound but crashes at start (16bit mm)

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

Summary: Pumuckl: regained sound but crashes at start (16bit
                    mm)
           Product: Wine
           Version: 1.1.33
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: winmm&mci
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
                CC: [e-mail]

Created an attachment (id=24979)
 --> (http://bugs.winehq.org/attachment.cgi?id=24979)
+winmm,+driver,+mmsys log and backtrace

Hi,

commit 8575935f610bafd3b3529d4243ca36b9a7fc6deb
Author: Eric Pouech 
Date:   Sun Nov 8 15:09:46 2009 +0100

mmsystem.dll16: Correctly send messages up to winmm.dll.
    Spotted by Christian Costa.

causes Pumuckl / Klabauterjagd to crash at program start in winevdm, during the
intro animation.
The patch is not that bad either, as it makes sound work again in
Klabauterjagd. Sound was lost somewhere between 1.1.30 IIRC and 1.1.33.

The app worked fine with older releases such as wine-1.1.18 (or 1.1.30 IIRC).

Should I run another round of git-bisect to determine the ancestor commit that
caused loss of sound?

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

Jeff Zaroyko  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

Jörg Höhle  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |win16

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

--- Comment #1 from Jörg Höhle   2010-01-19 11:38:15 ---
Still present in wine-1.1.36 which announced the completion of the 16-bit
separation, with both ALSA and OSS.

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

--- Comment #2 from Eric Pouech   2010-06-19 07:23:34 ---
can you check if the patch in #20816 fixes this issue ?
A+

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

Jörg Höhle  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Jörg Höhle   2010-06-23 04:59:40 ---
Fixed by commit 6f778a79f4f0e0c229d7dc7917a5fb5195dabd35
Thank you, Eric.

I'm submitting a patch that fixes the 9 bogus
fixme:winmm:MMSYSTDRV_SetHandle Already has a thunk for this handle (nil)!!!
which are caused by WAVE_FORMAT_QUERY in that app.

On the first run, Pumukl had one intermittent hang in a Win16Mutex and later
crashed when clicking the 'i' key in the keyboard menu.  Later runs (with no CD
in drive) worked perfectly with both OSS and ALSA in Ubuntu Intrepid.

[Bug 20848] Pumuckl: regained sound but crashes at start (16bit mm)

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard   2010-06-25 12:41:25 ---
Closing bugs fixed in 1.2-rc5.