diff options
author | Michael Niedermayer <[email protected]> | 2015-02-04 20:13:18 +0100 |
---|---|---|
committer | Michael Niedermayer <[email protected]> | 2015-02-13 20:48:08 +0100 |
commit | 3531ff8db31926c115ac771b3d5232261b1530f9 (patch) | |
tree | 3eb03c3af5a362ec927fb807568c692d1f5a3c07 | |
parent | ee8e48d38677225afc8903f7640ea1f48913cd47 (diff) |
avcodec/mjpegdec: Check escape sequence validity
Fixes assertion failure
Fixes: asan_heap-oob_1c1a4ea_1242_cov_2274415971_TESTcmyk.jpg
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <[email protected]>
(cherry picked from commit afa92907f3c6a0c3bdad766ec8d938ee17ee1c9e)
Signed-off-by: Michael Niedermayer <[email protected]>
-rw-r--r-- | libavcodec/mjpegdec.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index 95a066f3d6..e8bbf6080a 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -1877,6 +1877,10 @@ int ff_mjpeg_find_marker(MJpegDecodeContext *s, put_bits(&pb, 8, x); if (x == 0xFF) { x = src[b++]; + if (x & 0x80) { + av_log(s->avctx, AV_LOG_WARNING, "Invalid escape sequence\n"); + x &= 0x7f; + } put_bits(&pb, 7, x); bit_count--; } |