diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2014-10-28 15:26:42 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2015-03-12 18:03:48 +0100 |
| commit | 017226fdf9eb6e3985e87f6218d012f7b6fe99e3 (patch) | |
| tree | 20042a927c311e1f5af7e0772c4b3b556793db16 | |
| parent | 27d82d83741b317f90472c961ef81f38bdc86a9f (diff) | |
| download | ffmpeg-017226fdf9eb6e3985e87f6218d012f7b6fe99e3.tar.gz | |
avcodec/dxa: check dimensions
Fixes out of array access
Fixes: asan_heap-oob_11222fb_21_020.dxa
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit e70312dfc22c4e54d5716f28f28db8f99c74cc90)
Conflicts:
libavcodec/dxa.c
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
| -rw-r--r-- | libavcodec/dxa.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/libavcodec/dxa.c b/libavcodec/dxa.c index 20f5464e8a..a5bf1a051c 100644 --- a/libavcodec/dxa.c +++ b/libavcodec/dxa.c @@ -301,6 +301,11 @@ static av_cold int decode_init(AVCodecContext *avctx) c->avctx = avctx; avctx->pix_fmt = PIX_FMT_PAL8; + if (avctx->width%4 || avctx->height%4) { + av_log(avctx, AV_LOG_ERROR, "dimensions are not a multiple of 4"); + return AVERROR_INVALIDDATA; + } + avcodec_get_frame_defaults(&c->pic); avcodec_get_frame_defaults(&c->prev); |