[PATCH] mmdevapi/tests: Fix memory leaks (Coverity)
Andrew Eikum
aeikum at codeweavers.com
Wed Dec 20 07:27:50 CST 2017
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
On Tue, Dec 19, 2017 at 09:00:17AM -0700, Alex Henrie wrote:
> 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