mmdevapi/tests: Fix memory leaks (Coverity).

Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>
Signed-off-by: Andrew Eikum <aeikum@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Alex Henrie 2017-12-19 09:00:17 -07:00 committed by Alexandre Julliard
parent 68cff26b1b
commit 10aa228ccc
2 changed files with 4 additions and 8 deletions

View File

@ -556,8 +556,7 @@ static void test_audioclient(void)
if (hr != S_OK) if (hr != S_OK)
{ {
skip("Cannot initialize %08x, remainder of tests is useless\n", hr); skip("Cannot initialize %08x, remainder of tests is useless\n", hr);
CoTaskMemFree(pwfx); goto cleanup;
return;
} }
hr = IAudioClient_GetStreamLatency(ac, NULL); hr = IAudioClient_GetStreamLatency(ac, NULL);
@ -592,6 +591,7 @@ static void test_audioclient(void)
test_capture(ac, handle, pwfx); test_capture(ac, handle, pwfx);
cleanup:
IAudioClient_Release(ac); IAudioClient_Release(ac);
CloseHandle(handle); CloseHandle(handle);
CoTaskMemFree(pwfx); CoTaskMemFree(pwfx);

View File

@ -323,11 +323,7 @@ static void test_audioclient(void)
hr = IAudioClient_Initialize(ac, AUDCLNT_SHAREMODE_SHARED, 0, 5000000, 0, pwfx, NULL); hr = IAudioClient_Initialize(ac, AUDCLNT_SHAREMODE_SHARED, 0, 5000000, 0, pwfx, NULL);
ok(hr == S_OK, "Valid Initialize returns %08x\n", hr); ok(hr == S_OK, "Valid Initialize returns %08x\n", hr);
if (hr != S_OK) if (hr != S_OK)
{ goto cleanup;
IAudioClient_Release(ac);
CoTaskMemFree(pwfx);
return;
}
hr = IAudioClient_GetStreamLatency(ac, NULL); hr = IAudioClient_GetStreamLatency(ac, NULL);
ok(hr == E_POINTER, "GetStreamLatency(NULL) call returns %08x\n", hr); ok(hr == E_POINTER, "GetStreamLatency(NULL) call returns %08x\n", hr);
@ -372,8 +368,8 @@ static void test_audioclient(void)
hr = IAudioClient_Start(ac); hr = IAudioClient_Start(ac);
ok(hr == AUDCLNT_E_NOT_STOPPED, "Start twice returns %08x\n", hr); ok(hr == AUDCLNT_E_NOT_STOPPED, "Start twice returns %08x\n", hr);
cleanup:
IAudioClient_Release(ac); IAudioClient_Release(ac);
CloseHandle(handle); CloseHandle(handle);
CoTaskMemFree(pwfx); CoTaskMemFree(pwfx);
} }