Re: [PATCH v8 0/2] MR8459: urlmon: Add PersistentZoneIdentifier semi-stub

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/sec_mgr.c:
> -
> - remove_dir(tmp_dir);
> -
>
> +}
> -
>
> +static void test_PersistentZoneIdentifier(void)
> +{
> - IUnknown *unk;
> - IPersistFile *persist_file, *persist_file2;
> - IZoneIdentifier *zone_id, *zone_id2;
> - HRESULT hres;
> -
> - trace("Testing uninitialized state of PersistentZoneIdetifier...\n");
> -
> - hres = CoCreateInstance(&CLSID_PersistentZoneIdentifier, NULL,
> -
>    CLSCTX_INPROC_SERVER, &IID_IUnknown, (void**)&unk);

`unk` is leaked, it needs a corresponding `Release` call. The same is true for all interfaces returned by `QueryInterface`.