On 3/31/20 7:03 PM, Jactry Zeng wrote:
> + if (filter->filename)
> + {
> + CoTaskMemFree(filter->filename);
> + FreeMediaType(&filter->type);
> + }
> +
It should match the way 'filename' was allocated. Also, does it make
sense to check for null pointer at all?