[Bug 52998] New: xaudio2_7:xaudio2 fails on Windows 1909+
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(a)winehq.org Reporter: fgouget(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52998 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum(a)codeweavers.com Keywords| |source, testcase -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52998 --- Comment #1 from Andrew Eikum <aeikum(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52998 --- Comment #2 from Andrew Eikum <aeikum(a)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(a)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(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52998 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |c1e793f1119de0c0ef7d4bd6d9f | |efbafdb5dbbe5 Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #3 from François Gouget <fgouget(a)codeweavers.com> --- Yes, these failures are gone. Thanks. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52998 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 7.11. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla