aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2015-12-04 00:31:44 +0100
committerMichael Niedermayer <michael@niedermayer.cc>2015-12-04 01:55:11 +0100
commit77a644e6fa4aaeb2c26cfaa0e8ec3b19829b8d88 (patch)
treeabe60320facc3ca608da9ed3e02663bf66afb468
parentcafb19560401612a07760d230a50d9c1d0564daf (diff)
downloadffmpeg-77a644e6fa4aaeb2c26cfaa0e8ec3b19829b8d88.tar.gz
avcodec/h264_direct: Check CO-POCs before use
Fixes harmless integer overflow Fixes mozilla bug 1230286 Found-by: Tyson Smith Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavcodec/h264_direct.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/libavcodec/h264_direct.c b/libavcodec/h264_direct.c
index 5756a7ba66..5f66a67d89 100644
--- a/libavcodec/h264_direct.c
+++ b/libavcodec/h264_direct.c
@@ -137,6 +137,10 @@ void ff_h264_direct_ref_list_init(const H264Context *const h, H264SliceContext *
if (h->picture_structure == PICT_FRAME) {
int cur_poc = h->cur_pic_ptr->poc;
int *col_poc = sl->ref_list[1][0].parent->field_poc;
+ if (col_poc[0] == INT_MAX && col_poc[1] == INT_MAX) {
+ av_log(h->avctx, AV_LOG_ERROR, "co located POCs unavailable\n");
+ sl->col_parity = 1;
+ } else
sl->col_parity = (FFABS(col_poc[0] - cur_poc) >=
FFABS(col_poc[1] - cur_poc));
ref1sidx =