[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