[PATCH] windowscodecs/tests: Fix a memory leak (Valgrind).

Dmitry Timoshkov dmitry at baikal.ru
Sun Aug 18 21:51:16 CDT 2019


Sven Baars <sven.wine at gmail.com> wrote:

> diff --git a/dlls/windowscodecs/tests/metadata.c b/dlls/windowscodecs/tests/metadata.c
> index b434648fe5..a955702c82 100644
> --- a/dlls/windowscodecs/tests/metadata.c
> +++ b/dlls/windowscodecs/tests/metadata.c
> @@ -3029,6 +3029,7 @@ static void test_queryreader(void)
>                      ok(hr == E_INVALIDARG, "got %#x\n", hr);
>  
>                      IWICMetadataQueryReader_Release(new_reader);
> +                    PropVariantClear(&value);
>                  }
>                  else if (value.vt == VT_LPSTR)
>                      ok(!lstrcmpA(U(value).pszVal, test_data[i].str_value), "%u: expected %s, got %s\n",

Did you see a comment 7 lines lower? Calling PropVariantClear() for fake
types will lead to crashes.

-- 
Dmitry.



More information about the wine-devel mailing list