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

Ronald S. Bultje rsbultje at gmail.com
Wed Nov 13 19:25:40 EET 2019


Hi,

On Wed, Nov 13, 2019 at 2:00 AM <quinkblack at foxmail.com> wrote:

> From: Zhao Zhili <zhilizhao at tencent.com>
>
> vp7 decoder doesn't set update_thread_context field
> ---
>  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);
>

OK, I guess. What's the actual problem solved by this patch (e.g. some
commandline invocation with particular settings)? Does it cause some
assertion with frame-multithreaded VP7 decoding? Maybe add that to the
commit message so we know how to test this change.

Ronald


More information about the ffmpeg-devel mailing list