[Bug 11793] New: Hulk demo crashes in DSOUND

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

Summary: Hulk demo crashes in DSOUND
           Product: Wine
           Version: 0.9.56.
          Platform: PC
               URL: http://www.jeuxvideo.com/downloads/0000/00008022_demo.ht
                    m
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: directx-dsound
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=11055)
 --> (http://bugs.winehq.org/attachment.cgi?id=11055)
console output

The Hulk demo crashes with a dsound error (see the attached console output
below).
Disabling the sound in winecfg makes the demo working.

Joaopa

[Bug 11793] Hulk demo crashes in DSOUND

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

joaopa  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #1 from joaopa   2008-03-01 05:06:25 ---
Created an attachment (id=11056)
 --> (http://bugs.winehq.org/attachment.cgi?id=11056)
console output with +dsound,warn+dsound3d

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #2 from joaopa   2008-03-01 05:12:25 ---
Created an attachment (id=11057)
 --> (http://bugs.winehq.org/attachment.cgi?id=11057)
console output with dsound3d

[Bug 11793] Hulk demo crashes in DSOUND

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

Lei Zhang  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #3 from Lei Zhang   2008-03-01 15:16:12 ---
confirming

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #4 from Lei Zhang   2008-03-01 15:35:34 ---
The crash goes away with native dsound.dll. Not sure where that 0x18 pointer
value came from.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #5 from Saulius K.   2008-03-04 00:06:22 ---
One idea of where does 0x18 pointer value come from is dereferencing pointer
(to some struct) which has NULL value and then using ptr to smaller, embedded
struct DS3DBUFFER.

Maybe +relay trace (at least ~3000 lines before "wine: Unhandled page
fault...") would suffice for successful investigation?  Well, +snoop info may
be needed also, but I am not so sure.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #6 from joaopa   2008-03-04 03:50:20 ---
Created an attachment (id=11102)
 --> (http://bugs.winehq.org/attachment.cgi?id=11102)
consol;e output with +relay,+snoop channels

[Bug 11793] Hulk demo crashes in DSOUND

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

joaopa  changed:

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

--- Comment #7 from joaopa   2008-03-04 05:51:15 ---
Wahoo, it is a regression. I found the bad commit:

d322d63b10b4a71f4d71f43e83d0c1b4d20c7414 is first bad commit
commit d322d63b10b4a71f4d71f43e83d0c1b4d20c7414
Author: Maarten Lankhorst 
Date:   Mon Oct 8 20:11:13 2007 +0200

winmm: Set default sound driver to alsa,oss,coreaudio.

:040000 040000 5a2ffa95a7acf3a4a3dc007b8bfb5acf8ff5b97c
58bc0dd57a11c292381d5aa66972564d566f541a M    dlls
:040000 040000 2e16571543166dc4be2499e079cc6b66a7dd5e70
cfdd753db5fe02aa2438e4e647b89cdbafb9aea1 M    programs

Joaopa

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #8 from Lei Zhang   2008-03-04 18:52:01 ---
I don't think that's right. I went back to wine 0.9.46, before that commit, and
it still failed the same way.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #9 from Lei Zhang   2008-03-04 19:00:09 ---
If it is a regression, then what version of Wine did this game work in?

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #10 from joaopa   2008-03-04 23:21:46 ---
Very strange, now the demo crashes with wine-0.9.46.

But it works with wine-0.9.43. Could you confirm before I do a new regression 
test?

Joaopa

[Bug 11793] Hulk demo crashes in DSOUND

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

Lei Zhang  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #11 from Lei Zhang   2008-03-05 12:52:40 ---
Ah, it was a different crash with Wine 0.9.40. With Wine 0.9.43, it does not
crash, but there was no sound either.

Regression test between 0.9.43 and 0.9.46 says:

commit 9c469e1a9487d81186642282de89048fc51c0864
Author: Maarten Lankhorst 
Date:   Sun Aug 19 23:52:19 2007 +0200

dsound: Handle failing of IDsDriver_CreateSoundBuffer better.

:040000 040000 62e4ffb7970af6ff05301585b9357388fb01be90
0c6d8930b74556ddb59b8b358e3c33d23cf3c83e M      dlls

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #12 from Maarten Lankhorst   2008-06-15 08:43:26 ---
I see no reason why either patch would cause it, could it be a uninitialized
variable or heap corruption?

WINEDEBUG=warn+heap might find something

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #13 from joaopa   2008-06-15 09:33:44 ---
Created an attachment (id=14064)
 --> (http://bugs.winehq.org/attachment.cgi?id=14064)
The requested trace

[Bug 11793] Hulk demo crashes in DSOUND

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

joaopa  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14064|0                           |1
        is obsolete|                            |

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #14 from joaopa   2008-07-21 07:57:32 ---
Created an attachment (id=14955)
 --> (http://bugs.winehq.org/attachment.cgi?id=14955)
debug trace with WINEDEBUG=+dsound,+dsound3d,+snoop,+heap

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #15 from Fab   2008-12-12 05:13:51 ---
still crashing with wine 1.1.10 and an clean WINEPREFIX.

[Bug 11793] Hulk demo crashes in DSOUND

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

Benjamin Debski  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #16 from Benjamin Debski   2010-01-04 19:35:24 ---
Still preset in Wine 1.1.35

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #17 from Benjamin Debski   2010-04-17 19:41:40 ---
Still present in Wine 1.1.43

[Bug 11793] Hulk demo crashes in DSOUND

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

Raymond  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #18 from Raymond   2010-04-19 01:21:06 ---
(In reply to comment #12)
> I see no reason why either patch would cause it, could it be a uninitialized
> variable or heap corruption?
>
> WINEDEBUG=warn+heap might find something

did wine implement secondary buffer ?

trace:dsound:SecondaryBufferImpl_SetVolume (0x157f170,0)
trace:dsound:IDirectSoundBufferImpl_SetVolume (0x157efa8,0)
wine: Unhandled page fault on read access to 0x00000018 at address 0x7ddeeac2
(thread 0009), starting debugger...

[Bug 11793] Hulk demo crashes in DSOUND

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

joaopa  changed:

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

--- Comment #19 from joaopa   2010-07-29 06:35:20 ---
> did wine implement secondary buffer ?

Yes!!

Removed the keyword regression, since this bug is not one.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #20 from Raymond   2010-07-29 20:45:56 ---
(In reply to comment #5)
> One idea of where does 0x18 pointer value come from is dereferencing pointer
> (to some struct) which has NULL value and then using ptr to smaller, embedded
> struct DS3DBUFFER.
>
> Maybe +relay trace (at least ~3000 lines before "wine: Unhandled page
> fault...") would suffice for successful investigation?  Well, +snoop info may
> be needed also, but I am not so sure.

ds3d mean wine pan any mono sources to stereo device by software atten of left
and right channels or hardware volume controls

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #21 from Raymond   2010-07-29 21:01:05 ---
if your sound card support hardware mixing and there are still available
hardware buffers, ds3dcreatebuffer  should create a stereo secondary hardware
buffer and play to sound card ( setvolumepan using hardware mixing volume
control )

otherwise it create a stereo secondary software buffer  which mix with other
secondary software buffer to primary hardware buffer and play to sound card (
setvolumepan using software atten of left and right channels )

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #22 from joaopa   2010-07-31 07:19:55 ---
Raymond, do you have any idea to fix this bug?

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #23 from joaopa   2010-07-31 07:24:10 ---
My sound card is Intel Corporation|82801G (ICH7 Family) High Definition Audio
Controller

Dont know if it supports hardware mixing though

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #24 from Raymond   2010-07-31 20:45:53 ---
(In reply to comment #22)
> Raymond, do you have any idea to fix this bug?

Backtrace:
=>1 0x7d1abd12 IDirectSound3DBufferImpl_SetAllParameters+0x32(iface=0x157f138,
lpcDs3dBuffer=0x18, dwApply=0x1) [/home/david/wine/dlls/dsound/sound3d.c:486]
in dsound (0x0032f37c)
  2 0x005a4cc4 in hulk_demo (+0x1a4cc4) (0x00000000)
0x7d1abd12 IDirectSound3DBufferImpl_SetAllParameters+0x32
[/home/david/wine/dlls/dsound/sound3d.c:486] in dsound: movl    0x0(%esi),%ecx
486        if (lpcDs3dBuffer->dwSize != sizeof(DS3DBUFFER)) {

it is not easy now since "dsound_test.exe ds3d" fail with

fixme:dsound3d:IDirectSound3DListenerImpl_QueryInterface Unknown IID
{31efac30-515c-11d0-a9aa-00aa0061be93}
ds3d.c:1202: Test failed: IDirectSound3DListener_QueryInterface didn't handle
IKsPropertySet: ret = 80004002

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #25 from joaopa  2010-11-06 14:51:17 CDT ---
Created an attachment (id=31759)
 --> (http://bugs.winehq.org/attachment.cgi?id=31759)
valgrind output

I hope it helps

[Bug 11793] Hulk demo crashes in DSOUND

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

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #26 from Dan Kegel  2010-11-06 22:22:21 CDT ---
Your valgrind is too old, see 
http://wiki.winehq.org/Wine_and_Valgrind#head-6ab217df820e87e9b038fc54dfb50d...

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #27 from joaopa  2010-11-06 23:44:45 CDT ---
Created an attachment (id=31769)
 --> (http://bugs.winehq.org/attachment.cgi?id=31769)
valgrind output with valgrind 3.6.0

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #28 from joaopa  2011-05-13 02:02:55 CDT ---
bug still there in current git.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #29 from joaopa  2011-05-13 02:10:30 CDT ---
Using a native dsound.dll is a workaround for this bug.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #30 from joaopa  2011-05-14 15:29:01 CDT ---
Created an attachment (id=34707)
 --> (http://bugs.winehq.org/attachment.cgi?id=34707)
new valgrind output

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #31 from joaopa  2011-09-28 02:00:44 CDT ---
Bug still there in current git.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #32 from joaopa  2012-01-20 01:04:55 CST ---
bug still there in current git.

[Bug 11793] Hulk demo crashes in DSOUND

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

--- Comment #33 from joaopa  2012-05-15 01:35:27 CDT ---
bug still there in current git.

[Bug 11793] Hulk demo crashes in DSOUND

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

joaopa  changed:

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

--- Comment #34 from joaopa  2012-07-19 15:01:16 CDT ---
Great !! One of the dsound clean-up patch of today fixed this pretty old bug.
Thanks Mickael.

[Bug 11793] Hulk demo crashes in DSOUND

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

Alexandre Julliard  changed:

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

--- Comment #35 from Alexandre Julliard  2012-07-31 14:19:18 CDT ---
Closing bugs fixed in 1.5.10.