[FFmpeg-devel] [PATCH 1/3] avcodec/vp3: Move work after ff_thread_finish_setup

[Andreas Rheinhardt]

Andreas Rheinhardt:
> all_fragments is not synced between threads; resetting it can wait.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/vp3.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vp3.c b/libavcodec/vp3.c
> index 5002800ef2..272dd89117 100644
> --- a/libavcodec/vp3.c
> +++ b/libavcodec/vp3.c
> @@ -2757,9 +2757,9 @@ static int vp3_decode_frame(AVCodecContext *avctx, AVFrame *frame,
>              ff_thread_report_progress(&s->last_frame, INT_MAX, 0);
>          }
>      }
> +    ff_thread_finish_setup(avctx);
>  
>      memset(s->all_fragments, 0, s->fragment_count * sizeof(Vp3Fragment));
> -    ff_thread_finish_setup(avctx);
>  
>      if (s->version < 2) {
>          if ((ret = unpack_superblocks(s, &gb)) < 0) {

Will apply this patchset tomorrow unless there are objections.

- Andreas