[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