avcodec/pngdec: Check bits per pixel before setting monoblack pixel format
authorMichael Niedermayer <michaelni@gmx.at>
Fri, 3 Oct 2014 15:35:58 +0000 (17:35 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Wed, 11 Mar 2015 23:47:04 +0000 (00:47 +0100)
Fixes out of array accesses
Fixes: asan_heap-oob_14dbfcf_4_asan_heap-oob_1ce5767_179_add_method_small.png

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit 3e2b745020c2dbf0201fe7df3dad9e7e0b2e1bb6)

Conflicts:

libavcodec/pngdec.c

libavcodec/pngdec.c

index 9afa093..6a49530 100644 (file)
@@ -463,7 +463,7 @@ static int decode_frame(AVCodecContext *avctx,
                 } else if (s->bit_depth == 16 &&
                            s->color_type == PNG_COLOR_TYPE_RGB) {
                     avctx->pix_fmt = PIX_FMT_RGB48BE;
-                } else if (s->bit_depth == 1) {
+                } else if (s->bit_depth == 1 && s->bits_per_pixel == 1) {
                     avctx->pix_fmt = PIX_FMT_MONOBLACK;
                 } else if (s->bit_depth == 8 &&
                            s->color_type == PNG_COLOR_TYPE_PALETTE) {