> 在 2019年11月29日,13:39,Jeyapal, Karthick <kjeya...@akamai.com> 写道:
> 
> 
> On 11/29/19 11:08 AM, Steven Liu wrote:
>> 
>> 
>>> 在 2019年11月29日,13:35,Jeyapal, Karthick <kjeya...@akamai.com> 写道:
>>> 
>>> 
>>> On 11/29/19 10:45 AM, Steven Liu wrote:
>>>> Signed-off-by: Steven Liu <l...@chinaffmpeg.org>
>>>> ---
>>>> libavformat/dashenc.c | 3 +--
>>>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>>> 
>>>> diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
>>>> index a462876c13..8c28fb6b6e 100644
>>>> --- a/libavformat/dashenc.c
>>>> +++ b/libavformat/dashenc.c
>>>> @@ -588,8 +588,7 @@ static void dash_free(AVFormatContext *s)
>>>>                avio_close(os->ctx->pb);
>>>>        }
>>>>        ff_format_io_close(s, &os->out);
>>>> -        if (os->ctx)
>>>> -            avformat_free_context(os->ctx);
>>>> +        avformat_free_context(os->ctx);
>>> This 'if' condition is a safety net against double frees caused due to any 
>>> corner case and/or wrong usage.
>>> Unless this is absolutely required for some reason, this 'if' condition 
>>> should not be removed.
>> 
>> 
>> 4433 void avformat_free_context(AVFormatContext *s)
>> 4434 {
>> 4435     int i;
>> 4436
>> 4437     if (!s)
>> 4438         return;
>> 
>> This check is in the function of avformat_free_context.
> Great! Then it is fine. You can remove it.
Thanks Jeyapal :)
>> 
>>>>        for (j = 0; j < os->nb_segments; j++)
>>>>            av_free(os->segments[j]);
>>>>        av_free(os->segments);
>>> 
>> 
>> Thanks
>> Steven

Thanks
Steven





_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to