[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH 3/4] qasf: Partially implement IFileSourceFilter_Load() for WM ASF reader.
From:       Nikolay Sivov <nsivov () codeweavers ! com>
Date:       2020-03-31 18:25:28
Message-ID: 16e9f437-947a-c16d-7d1d-882bdce5cc1e () codeweavers ! com
[Download RAW message or body]



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?

[prev in list] [next in list] [prev in thread] [next in thread]