diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-12-19 21:59:42 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-12-19 22:19:21 +0100 |
| commit | 3a3be0220834b1bb1de2e68da47cc944cbf5f746 (patch) | |
| tree | 8afac214a9657596ac8adf9b9f56776ace86fa63 | |
| parent | cabd9ae5be8797bbafb8db15be5edd266d9efd7a (diff) | |
| download | ffmpeg-3a3be0220834b1bb1de2e68da47cc944cbf5f746.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 a81ee76cfd..a3de6b2f35 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -303,7 +303,7 @@ int ff_h264_decode_ref_pic_list_reordering(H264Context *h, H264SliceContext *sl) 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; |