[FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: Don't attach FrameDecodeData unnecessarily

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Fri May 13 20:07:52 EEST 2022


Andreas Rheinhardt:
> It is unneeded, as this decoder does not call ff_get_buffer().
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/wrapped_avframe.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/libavcodec/wrapped_avframe.c b/libavcodec/wrapped_avframe.c
> index 3af007d478..06c274eed0 100644
> --- a/libavcodec/wrapped_avframe.c
> +++ b/libavcodec/wrapped_avframe.c
> @@ -98,12 +98,6 @@ static int wrapped_avframe_decode(AVCodecContext *avctx, AVFrame *out,
>  
>      av_frame_move_ref(out, in);
>  
> -    err = ff_attach_decode_data(out);
> -    if (err < 0) {
> -        av_frame_unref(out);
> -        return err;
> -    }
> -
>      *got_frame = 1;
>      return 0;
>  }

Will apply this patch tomorrow.
And I'd really hope that someone can review and/or test patch 2/2.

- Andreas


More information about the ffmpeg-devel mailing list