Commit 6060234d authored by Michael Niedermayer's avatar Michael Niedermayer

avcodec/mjpegdec: check len in mjpeg_decode_app() more completely

Avoids len from becoming negative and causing assertion failure

Fixes: signal_sigabrt_7ffff7126425_5140_fd44dc63fa7bdd12ee34fc602231ef02.jpg

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parent ba992711
...@@ -1443,7 +1443,7 @@ static int mjpeg_decode_app(MJpegDecodeContext *s) ...@@ -1443,7 +1443,7 @@ static int mjpeg_decode_app(MJpegDecodeContext *s)
int len, id, i; int len, id, i;
len = get_bits(&s->gb, 16); len = get_bits(&s->gb, 16);
if (len < 5) if (len < 6)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
if (8 * len > get_bits_left(&s->gb)) if (8 * len > get_bits_left(&s->gb))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
...@@ -1557,7 +1557,7 @@ static int mjpeg_decode_app(MJpegDecodeContext *s) ...@@ -1557,7 +1557,7 @@ static int mjpeg_decode_app(MJpegDecodeContext *s)
} }
/* EXIF metadata */ /* EXIF metadata */
if (s->start_code == APP1 && id == AV_RB32("Exif")) { if (s->start_code == APP1 && id == AV_RB32("Exif") && len >= 2) {
GetByteContext gbytes; GetByteContext gbytes;
int ret, le, ifd_offset, bytes_read; int ret, le, ifd_offset, bytes_read;
const uint8_t *aligned; const uint8_t *aligned;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment