From a26e9c1040afeecf9013da742b0dec7009445f2b Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Fri, 24 Jan 2014 03:24:55 +0100 Subject: [PATCH] avcodec/mjpegenc: Use av_frame_clone() instead of av_frame_ref() This avoids the need for double error checking Fixes CID1163843 Signed-off-by: Michael Niedermayer --- libavcodec/mjpegenc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/mjpegenc.c b/libavcodec/mjpegenc.c index 44e1c3c76d..7487ef8f66 100644 --- a/libavcodec/mjpegenc.c +++ b/libavcodec/mjpegenc.c @@ -539,10 +539,9 @@ static int amv_encode_picture(AVCodecContext *avctx, AVPacket *pkt, if(s->avctx->flags & CODEC_FLAG_EMU_EDGE) return AVERROR(EINVAL); - pic = av_frame_alloc(); + pic = av_frame_clone(pic_arg); if (!pic) return AVERROR(ENOMEM); - av_frame_ref(pic, pic_arg); //picture should be flipped upside-down for(i=0; i < 3; i++) { int vsample = i ? 2 >> chroma_v_shift : 2;