dsound: Reject WAVEFORMATEX formats with more than two channels.

Formats with more than two channels require WAVEFORMATEXTENSIBLE according to tests. Fix Viking: Battle for Asgard (211160) audio cracking in its intro video.

dsound: Reject WAVEFORMATEX formats with more than two channels.
7c7b2e8e · Zhiyi Zhang · Alexandre Julliard · b3ec5bc7 · 7c7b2e8e · 7c7b2e8e
Commit 7c7b2e8e authored Feb 20, 2024 by Zhiyi Zhang Committed by Alexandre Julliard Feb 20, 2024
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

dsound.c dlls/dsound/dsound.c +3 -0

primary.c dlls/dsound/primary.c +3 -0

dsound.c dlls/dsound/tests/dsound.c +0 -2

No files found.
--- a/dlls/dsound/dsound.c
+++ b/dlls/dsound/dsound.c
@@ -490,6 +490,9 @@ static HRESULT DirectSoundDevice_CreateSoundBuffer(
            return DSERR_INVALIDPARAM;
        }

+        if (dsbd->lpwfxFormat->nChannels > 2 && dsbd->lpwfxFormat->wFormatTag != WAVE_FORMAT_EXTENSIBLE)
+            return DSERR_INVALIDPARAM;
+
        if (dsbd->lpwfxFormat->wFormatTag == WAVE_FORMAT_EXTENSIBLE)
        {
            WAVEFORMATEXTENSIBLE *pwfxe = (WAVEFORMATEXTENSIBLE*)dsbd->lpwfxFormat;

--- a/dlls/dsound/primary.c
+++ b/dlls/dsound/primary.c
@@ -465,6 +465,9 @@ HRESULT primarybuffer_SetFormat(DirectSoundDevice *device, LPCWAVEFORMATEX passe
 			return DSERR_INVALIDPARAM;
 	}

+        if (passed_fmt->nChannels > 2 && passed_fmt->wFormatTag != WAVE_FORMAT_EXTENSIBLE)
+            return DSERR_ALLOCATED;
+
 	/* **** */
 	AcquireSRWLockExclusive(&device->buffer_list_lock);
 	EnterCriticalSection(&(device->mixlock));

--- a/dlls/dsound/tests/dsound.c
+++ b/dlls/dsound/tests/dsound.c
@@ -1500,7 +1500,6 @@ static void perform_invalid_fmt_tests(const char *testname, IDirectSound *dso, I
    wfx.nBlockAlign = wfx.nChannels * wfx.wBitsPerSample / 8;
    wfx.nAvgBytesPerSec = wfx.nSamplesPerSec * wfx.nBlockAlign;
    rc = do_invalid_fmt_test(dso, buf, &wfx, &got_buf);
-    todo_wine
    ok(rc == (buf ? DSERR_ALLOCATED : DSERR_INVALIDPARAM), "%s: SetFormat: %08lx\n", testname, rc);

    wfx.wFormatTag = WAVE_FORMAT_PCM;
@@ -1510,7 +1509,6 @@ static void perform_invalid_fmt_tests(const char *testname, IDirectSound *dso, I
    wfx.nBlockAlign = wfx.nChannels * wfx.wBitsPerSample / 8;
    wfx.nAvgBytesPerSec = wfx.nSamplesPerSec * wfx.nBlockAlign;
    rc = do_invalid_fmt_test(dso, buf, &wfx, &got_buf);
-    todo_wine
    ok(rc == (buf ? DSERR_ALLOCATED : DSERR_INVALIDPARAM), "%s: SetFormat: %08lx\n", testname, rc);

    fmtex.Format.cbSize = sizeof(WAVEFORMATEXTENSIBLE) - sizeof(WAVEFORMATEX);