[PATCH] mmdevapi/tests: Fix memory leaks (Coverity)
Alex Henrie
alexhenrie24 at gmail.com
Tue Dec 19 10:00:17 CST 2017
Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
dlls/mmdevapi/tests/capture.c | 4 ++--
dlls/mmdevapi/tests/render.c | 8 ++------
2 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/dlls/mmdevapi/tests/capture.c b/dlls/mmdevapi/tests/capture.c
index 99da296e0a..3cdfda1a37 100644
--- a/dlls/mmdevapi/tests/capture.c
+++ b/dlls/mmdevapi/tests/capture.c
@@ -556,8 +556,7 @@ static void test_audioclient(void)
if (hr != S_OK)
{
skip("Cannot initialize %08x, remainder of tests is useless\n", hr);
- CoTaskMemFree(pwfx);
- return;
+ goto cleanup;
}
hr = IAudioClient_GetStreamLatency(ac, NULL);
@@ -592,6 +591,7 @@ static void test_audioclient(void)
test_capture(ac, handle, pwfx);
+cleanup:
IAudioClient_Release(ac);
CloseHandle(handle);
CoTaskMemFree(pwfx);
diff --git a/dlls/mmdevapi/tests/render.c b/dlls/mmdevapi/tests/render.c
index e394d59cfe..9d3e8617b9 100644
--- a/dlls/mmdevapi/tests/render.c
+++ b/dlls/mmdevapi/tests/render.c
@@ -323,11 +323,7 @@ static void test_audioclient(void)
hr = IAudioClient_Initialize(ac, AUDCLNT_SHAREMODE_SHARED, 0, 5000000, 0, pwfx, NULL);
ok(hr == S_OK, "Valid Initialize returns %08x\n", hr);
if (hr != S_OK)
- {
- IAudioClient_Release(ac);
- CoTaskMemFree(pwfx);
- return;
- }
+ goto cleanup;
hr = IAudioClient_GetStreamLatency(ac, NULL);
ok(hr == E_POINTER, "GetStreamLatency(NULL) call returns %08x\n", hr);
@@ -372,8 +368,8 @@ static void test_audioclient(void)
hr = IAudioClient_Start(ac);
ok(hr == AUDCLNT_E_NOT_STOPPED, "Start twice returns %08x\n", hr);
+cleanup:
IAudioClient_Release(ac);
-
CloseHandle(handle);
CoTaskMemFree(pwfx);
}
--
2.15.1
More information about the wine-devel
mailing list