[FFmpeg-devel] [PATCH 06/13] avcodec/roqvideodec: Cleanup generically after init failure

[Paul B Mahol]

On 8/29/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavcodec/roqvideodec.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/roqvideodec.c b/libavcodec/roqvideodec.c
> index a0c293f2f0..dd045ed5eb 100644
> --- a/libavcodec/roqvideodec.c
> +++ b/libavcodec/roqvideodec.c
> @@ -184,11 +184,8 @@ static av_cold int roq_decode_init(AVCodecContext
> *avctx)
>
>      s->last_frame    = av_frame_alloc();
>      s->current_frame = av_frame_alloc();
> -    if (!s->current_frame || !s->last_frame) {
> -        av_frame_free(&s->current_frame);
> -        av_frame_free(&s->last_frame);
> +    if (!s->current_frame || !s->last_frame)
>          return AVERROR(ENOMEM);
> -    }
>
>      avctx->pix_fmt = AV_PIX_FMT_YUVJ444P;
>      avctx->color_range = AVCOL_RANGE_JPEG;
> @@ -248,4 +245,5 @@ AVCodec ff_roq_decoder = {
>      .close          = roq_decode_end,
>      .decode         = roq_decode_frame,
>      .capabilities   = AV_CODEC_CAP_DR1,
> +    .caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
>  };
> --
> 2.20.1
>

LGTM

> _______________________________________________
> 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".