avcodec/mjpegenc: Use av_frame_clone() instead of av_frame_ref()
authorMichael Niedermayer <michaelni@gmx.at>
Fri, 24 Jan 2014 02:24:55 +0000 (03:24 +0100)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 24 Jan 2014 02:24:55 +0000 (03:24 +0100)
This avoids the need for double error checking
Fixes CID1163843

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavcodec/mjpegenc.c

index 44e1c3c..7487ef8 100644 (file)
@@ -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;