[FFmpeg-cvslog] cdgraphics: do not return 0 from the decode function

Anton Khirnov git at videolan.org
Sun Aug 10 23:33:05 CEST 2014


ffmpeg | branch: release/2.3 | Anton Khirnov <anton at khirnov.net> | Wed Aug  6 10:56:34 2014 +0000| [a72b7286e6db0f5ffac386bb13fe6efe55fb0b12] | committer: Michael Niedermayer

cdgraphics: do not return 0 from the decode function

0 means no data consumed, so it can trigger an infinite loop in the
caller.

CC:libav-stable at libav.org
(cherry picked from commit c7d9b473e28238d4a4ef1b7e8b42c1cca256da36)

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a72b7286e6db0f5ffac386bb13fe6efe55fb0b12
---

 libavcodec/cdgraphics.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c
index b7a8fa7..3c07fc7 100644
--- a/libavcodec/cdgraphics.c
+++ b/libavcodec/cdgraphics.c
@@ -353,10 +353,9 @@ static int cdg_decode_frame(AVCodecContext *avctx,
         *got_frame = 1;
     } else {
         *got_frame = 0;
-        buf_size   = 0;
     }
 
-    return buf_size;
+    return avpkt->size;
 }
 
 static av_cold int cdg_decode_end(AVCodecContext *avctx)



More information about the ffmpeg-cvslog mailing list