> On Sep 10, 2020, at 8:57 AM, zhilizhao <quinkbl...@foxmail.com> wrote:
> 
> 
> 
>> On Aug 28, 2020, at 8:55 AM, Steven Liu <lingjiujia...@gmail.com> wrote:
>> 
>> <quinkbl...@foxmail.com> 于2020年8月27日周四 下午5:39写道:
>>> 
>>> From: Zhao Zhili <zhiliz...@tencent.com>
>>> 
>>> The lock is used in clear_frame_queue().
>>> ---
>>> libavcodec/videotoolboxenc.c | 11 +++++++----
>>> 1 file changed, 7 insertions(+), 4 deletions(-)
>>> 
>>> diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c
>>> index 988782f10d..758db9641f 100644
>>> --- a/libavcodec/videotoolboxenc.c
>>> +++ b/libavcodec/videotoolboxenc.c
>>> @@ -2496,14 +2496,17 @@ static av_cold int vtenc_close(AVCodecContext 
>>> *avctx)
>>> {
>>>    VTEncContext *vtctx = avctx->priv_data;
>>> 
>>> -    pthread_cond_destroy(&vtctx->cv_sample_sent);
>>> -    pthread_mutex_destroy(&vtctx->lock);
>>> -
>>> -    if(!vtctx->session) return 0;
>>> +    if(!vtctx->session) {
>>> +        pthread_cond_destroy(&vtctx->cv_sample_sent);
>>> +        pthread_mutex_destroy(&vtctx->lock);
>>> +        return 0;
>>> +    }
>>> 
>>>    VTCompressionSessionCompleteFrames(vtctx->session,
>>>                                       kCMTimeIndefinite);
>>>    clear_frame_queue(vtctx);
>>> +    pthread_cond_destroy(&vtctx->cv_sample_sent);
>>> +    pthread_mutex_destroy(&vtctx->lock);
>>>    CFRelease(vtctx->session);
>>>    vtctx->session = NULL;
>>> 
>>> --
>>> 2.28.0
>>> 
>>> _______________________________________________
>>> 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".
>> 
>> 
>> This patch look better than
>> https://patchwork.ffmpeg.org/project/ffmpeg/patch/20200827025327.28334-1...@chinaffmpeg.org/
>> 
> 
> Ping for the patch set.
Can you provide steps to reproduce the deadlock issue? I can get it pushed this 
weekend if so. 
> 
>> 
>> 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".
> 
> _______________________________________________
> 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".
_______________________________________________
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