[FFmpeg-devel] [PATCH v2] avcodec/vp8: fix multiple ff_thread_finish_setup() calls

[zhilizhao]

Ping for review, thanks!

> On Nov 14, 2019, at 12:29 PM, quinkblack at foxmail.com wrote:
> 
> From: Zhao Zhili <zhilizhao at tencent.com>
> 
> webp decoder doesn't set update_thread_context field
> 
> $ ffmpeg -i rgb_q80.webp -f null -
> [webp @ 0x7ffbd5823200] Multiple ff_thread_finish_setup() calls
> ---
> libavcodec/vp8.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c
> index eb51d1f3c9..b4deb3ed67 100644
> --- a/libavcodec/vp8.c
> +++ b/libavcodec/vp8.c
> @@ -2715,7 +2715,8 @@ int vp78_decode_frame(AVCodecContext *avctx, void *data, int *got_frame,
> 
>     s->next_framep[VP56_FRAME_CURRENT] = curframe;
> 
> -    ff_thread_finish_setup(avctx);
> +    if (avctx->codec->update_thread_context)
> +        ff_thread_finish_setup(avctx);
> 
>     if (avctx->hwaccel) {
>         ret = avctx->hwaccel->start_frame(avctx, avpkt->data, avpkt->size);
> -- 
> 2.22.0
> 
> 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".