diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-12-19 21:59:42 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-12-20 10:59:28 +0100 |
| commit | d0d9fad2ba5a0d53f59f69f780e3fb6f8ad97dff (patch) | |
| tree | 74c62e59c56d1bb3f3093cdd440e81d70e7e4862 | |
| parent | 3f7bbb0ded3f16c9c8c5de96bbc1b71b6979274b (diff) | |
| download | ffmpeg-d0d9fad2ba5a0d53f59f69f780e3fb6f8ad97dff.tar.gz | |
avcodec/h264_refs: Fix long_idx check
Fixes out of array read
Fixes mozilla bug 1233606
Found-by: Tyson Smith
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit b92b4775a0d07cacfdd2b4be6511f3cb362c977b)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/h264_refs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 826b2ca900..353412d27a 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -276,7 +276,7 @@ int ff_h264_decode_ref_pic_list_reordering(H264Context *h) long_idx = pic_num_extract(h, pic_id, &pic_structure); - if (long_idx > 31) { + if (long_idx > 31U) { av_log(h->avctx, AV_LOG_ERROR, "long_term_pic_idx overflow\n"); return AVERROR_INVALIDDATA; |