[Bug 52638] New: The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

Bug ID: 52638
           Summary: The 64-bit winmm:mci fails and crashes in
                    test_openCloseWAVE() in Wine
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: winmm&mci
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

The 64-bit winmm:mci crashes in test_openCloseWAVE() in Wine:

mci.c:389: Test failed: mci sysinfo all quantity returned
MCIERR_UNRECOGNIZED_COMMAND
mci.c:398: Test failed: sysinfo all quantity string and command differ
Unhandled exception: page fault on read access to 0x000000000000020d in 64-bit
code (0x00000000709b9ca1).
[...]
Backtrace:
=>0 0x00000000709b9ca1 _wcsicmp_l+0x91(str1=*** Invalid address
0x0000000000000002 ***
Internal symbol error: unable to access memory location 0000000000000002,
str2=*** Invalid address 0x0000000000000000 ***
Internal symbol error: unable to access memory location 0000000000000000,
locale=000000000064E980) [Z:\home\winetest\winetest\src\dlls\msvcrt\wcs.c:132]
in ucrtbase (0x000000000004dd50)
  1 0x00000000709b9d6c _wcsicmp+0xc(str1=<internal error>, str2=<internal
error>) [Z:\home\winetest\winetest\src\dlls\msvcrt\wcs.c:158] in ucrtbase
(0x00000000709b9d60)
  2 0x0000000000bf8057 MCI_FindCommand+0x38(s2=<internal error>, s1=<internal
error>) [Z:\home\winetest\winetest\src\dlls\winmm\mci.c:946] in winmm
(0x00000000709b9d60)
  3 0x0000000000bf8057 MCI_FindCommand+0x67(uTbl=<internal error>,
verb=<register RDI not accessible in this frame>)
[Z:\home\winetest\winetest\src\dlls\winmm\mci.c:938] in winmm
(0x00000000709b9d60)
  4 0x0000000000bfbcc7 mciSendStringW+0xc57(lpstrCommand=<internal error>,
lpstrRet=[<register RSP not accessible in this frame>, uRetLen=[<register RSP
not accessible in this frame>, hwndCallback=[<register RSP not accessible in
this frame>) [Z:\home\winetest\winetest\src\dlls\winmm\mci.c:1460] in winmm
(0x00000000003e0000)

https://test.winehq.org/data/patterns.html#winmm:mci

This only happens with the 64-bit build but as far as I can tell it is
systematic and impacts all machines.

A bisect pointed to the following commit as the trigger:

commit 7c76aa05a9d27dd6f5705c4ef90c16862af14f3e
Author: Jacek Caban 
Date:   Mon Mar 7 14:42:32 2022 +0100

win32u: Move class management implementation from user32.

Signed-off-by: Jacek Caban 
    Signed-off-by: Huw Davies 
    Signed-off-by: Alexandre Julliard

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression, source,
                   |                            |testcase
    Regression SHA1|                            |7c76aa05a9d27dd6f5705c4ef90
                   |                            |c16862af14f3e

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

Jacek Caban  changed:

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

--- Comment #1 from Jacek Caban  ---
I can't reproduce it here, could you please attach seh,class,win,server logs?

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

--- Comment #2 from François Gouget  ---
Created attachment 72011
  --> https://bugs.winehq.org/attachment.cgi?id=72011
WINEDEBUG=seh,class,win,server log

Note that the crash only happens when running winmm:mci through winetest.exe.
So this crashes:

./wine64 programs/winetest/winetest.exe -c -o - -t foo winmm:mci

But this does not:

./wine64 dlls//winmm/tests/winmm_test.exe mci

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

--- Comment #3 from Jacek Caban  ---
I can't reproduce it with winetest.exe neither and I don't see what's wrong
from those logs. Could you please attach a log with additional relay debug
channel? Maybe also warn+heap, since it may be some memory corruption.

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Fixed by SHA1|                            |f1d1d88c9e3b199c1d706073a7c
                   |                            |8afb9d4a2c2d0
         Resolution|---                         |FIXED

--- Comment #4 from Jacek Caban  ---
Fixed in git.

https://source.winehq.org/git/wine.git/commitdiff/f1d1d88c9e3b199c1d706073a7...

[Bug 52638] The 64-bit winmm:mci fails and crashes in test_openCloseWAVE() in Wine

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #5 from Alexandre Julliard  ---
Closing bugs fixed in 7.6.