[FFmpeg-cvslog] dfa: check that the caller set width/height properly.

Anton Khirnov git at videolan.org
Mon Feb 18 01:09:43 CET 2013


ffmpeg | branch: release/0.7 | Anton Khirnov <anton at khirnov.net> | Fri Sep 28 14:47:56 2012 +0200| [4c849c69910c6c59ba57022ef28aaf70192a0e12] | committer: Reinhard Tartler

dfa: check that the caller set width/height properly.

Fixes CVE-2012-2786.
(cherry picked from commit ee715f49a06bf3898246d01b056284a9bb1bcbb9)

Signed-off-by: Reinhard Tartler <siretart at tauware.de>

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

 libavcodec/dfa.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libavcodec/dfa.c b/libavcodec/dfa.c
index 919375b..1cb97dc 100644
--- a/libavcodec/dfa.c
+++ b/libavcodec/dfa.c
@@ -23,6 +23,8 @@
 #include "avcodec.h"
 #include "libavutil/intreadwrite.h"
 #include "bytestream.h"
+
+#include "libavutil/imgutils.h"
 #include "libavutil/lzo.h" // for av_memcpy_backptr
 
 typedef struct DfaContext {
@@ -35,9 +37,13 @@ typedef struct DfaContext {
 static av_cold int dfa_decode_init(AVCodecContext *avctx)
 {
     DfaContext *s = avctx->priv_data;
+    int ret;
 
     avctx->pix_fmt = PIX_FMT_PAL8;
 
+    if ((ret = av_image_check_size(avctx->width, avctx->height, 0, avctx)) < 0)
+        return ret;
+
     s->frame_buf = av_mallocz(avctx->width * avctx->height + AV_LZO_OUTPUT_PADDING);
     if (!s->frame_buf)
         return AVERROR(ENOMEM);



More information about the ffmpeg-cvslog mailing list