[FFmpeg-cvslog] cljr: Check if width or height are positive integers

Shitiz Garg git at videolan.org
Fri Dec 16 21:00:14 CET 2011


ffmpeg | branch: master | Shitiz Garg <mail-5deEcKV3Bynk1uMJSBkQmQ at public.gmane.org> | Fri Dec 16 05:08:21 2011 +0530| [4af0262f7d531c33b00d7f9dbca808d9c62d6a84] | committer: Michael Niedermayer

cljr: Check if width or height are positive integers

width and height might get passed as 0 and would cause floating point
exceptions in decode_frame.
Fixes bugzilla #149

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

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

 libavcodec/cljr.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/libavcodec/cljr.c b/libavcodec/cljr.c
index 7ae3e7e..2b0142b 100644
--- a/libavcodec/cljr.c
+++ b/libavcodec/cljr.c
@@ -61,6 +61,11 @@ static int decode_frame(AVCodecContext *avctx,
     if (p->data[0])
         avctx->release_buffer(avctx, p);
 
+    if (avctx->height <= 0 || avctx->width <= 0) {
+        av_log(avctx, AV_LOG_ERROR, "Invalid width or height\n");
+        return AVERROR_INVALIDDATA;
+    }
+
     if (buf_size / avctx->height < avctx->width) {
         av_log(avctx, AV_LOG_ERROR,
                "Resolution larger than buffer size. Invalid header?\n");



More information about the ffmpeg-cvslog mailing list