https://bugs.winehq.org/show_bug.cgi?id=52998
Bug ID: 52998 Summary: xaudio2_7:xaudio2 fails on Windows 1909+ Product: Wine Version: unspecified Hardware: x86-64 OS: Windows Status: NEW Severity: normal Priority: P2 Component: xaudio2 Assignee: wine-bugs@winehq.org Reporter: fgouget@codeweavers.com
xaudio2_7:xaudio2 has the same set of systematic failures on all Windows 1909+ versions:
xaudio2.c:1096: Test failed: xaudio2_8.dll: CreateFX({a90bc001-e897-e897-7439-435500000000}) failed: 8007000e xaudio2.c:1096: Test failed: xaudio2_8.dll: CreateFX({a90bc001-e897-e897-7439-435500000001}) failed: 8007000e xaudio2.c:1096: Test failed: xaudio2_8.dll: CreateFX({a90bc001-e897-e897-7439-435500000002}) failed: 8007000e xaudio2.c:1096: Test failed: xaudio2_8.dll: CreateFX({a90bc001-e897-e897-7439-435500000003}) failed: 8007000e xaudio2.c:1116: Test failed: xaudio2_8.dll: CreateFX(CLSID_AudioVolumeMeter) failed: 8007000e
https://test.winehq.org/data/patterns.html#xaudio2_7:xaudio2
The error code is E_OUTOFMEMORY and the failures correspond to the following formats: CLSID_FXEQ27 CLSID_FXMasteringLimiter27 CLSID_FXReverb27 CLSID_AudioVolumeMeter
Strangely enough the test succeeds on Windows 10 1507, 1607, 1709 and 1809 without any visible skip. So it's as if support for these formats was removed (but that's a strange error code to say so), or something more must be done to get them initialized properly.
https://bugs.winehq.org/show_bug.cgi?id=52998
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum@codeweavers.com Keywords| |source, testcase
https://bugs.winehq.org/show_bug.cgi?id=52998
--- Comment #1 from Andrew Eikum aeikum@codeweavers.com --- XAudio2 changed the CLSIDs for these effects between 2.7 and 2.8. I was curious to know if the old CLSIDs were supported by the newer versions, so I wrote those tests. I was surprised that they were supported, and I don't think any application actually used them in that way.
It's a little interesting that they dropped support for the legacy CLSIDs, but I guess it's good evidence that no application actually used that. We can just remove those tests, I think.
I submitted a patch to testbot, which passes: https://testbot.winehq.org/JobDetails.pl?Key=114787
https://bugs.winehq.org/show_bug.cgi?id=52998
--- Comment #2 from Andrew Eikum aeikum@codeweavers.com --- I sent in that patch to remove these tests, so I think this should be fixed going forward. Can you confirm that it's looking good now, François?
commit c1e793f1119de0c0ef7d4bd6d9fefbafdb5dbbe5 Author: Andrew Eikum aeikum@codeweavers.com Date: Wed Jun 1 13:06:28 2022 -0500
xaudio2/tests: Remove failing legacy CLSID tests.
Modern Windows fails these tests, and no application is known to need this behavior.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52998 Signed-off-by: Andrew Eikum aeikum@codeweavers.com Signed-off-by: Alexandre Julliard julliard@winehq.org
https://bugs.winehq.org/show_bug.cgi?id=52998
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |c1e793f1119de0c0ef7d4bd6d9f | |efbafdb5dbbe5 Status|NEW |RESOLVED Resolution|--- |FIXED
--- Comment #3 from François Gouget fgouget@codeweavers.com --- Yes, these failures are gone. Thanks.
https://bugs.winehq.org/show_bug.cgi?id=52998
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #4 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 7.11.
-
WineHQ Bugzilla