[FFmpeg-cvslog] smacker: check buffer size before reading output size

Justin Ruggles git at videolan.org
Mon Mar 19 05:30:41 CET 2012


ffmpeg | branch: release/0.8 | Justin Ruggles <justin.ruggles at gmail.com> | Wed Sep 21 11:42:55 2011 -0400| [b3d7fffee3bf0e2b28ce98ba609b7fd2099938db] | committer: Reinhard Tartler

smacker: check buffer size before reading output size

(cherry picked from commit cf044f8bff0d28dbc34492f18b0d18b3ba8bad9d)

Signed-off-by: Anton Khirnov <anton at khirnov.net>

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

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

diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c
index 33ea7b9..f3c59da 100644
--- a/libavcodec/smacker.c
+++ b/libavcodec/smacker.c
@@ -586,6 +586,11 @@ static int smka_decode_frame(AVCodecContext *avctx, void *data, int *data_size,
     int bits, stereo;
     int pred[2] = {0, 0};
 
+    if (buf_size <= 4) {
+        av_log(avctx, AV_LOG_ERROR, "packet is too small\n");
+        return AVERROR(EINVAL);
+    }
+
     unp_size = AV_RL32(buf);
 
     init_get_bits(&gb, buf + 4, (buf_size - 4) * 8);



More information about the ffmpeg-cvslog mailing list