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?