aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2015-09-03 09:20:23 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2015-11-26 16:58:32 +0100
commit4531c0dc8fe13128702972fae7b355f3bfdcf058 (patch)
treeac2a9b571ae88625bfc792d8bf4afa4d820a0336
parent8ebeefecf71ed6dbc285a240f9852077086a06d3 (diff)
downloadffmpeg-4531c0dc8fe13128702972fae7b355f3bfdcf058.tar.gz
avformat/mov: Fix integer overflow in FFABS
Fixes: unknown_unknown_19e_414_cov_764838672_bellhamlam.mov Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit 053e80f6eaf8d87521fe58ea96886b6ee0bbe59d) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavformat/mov.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c
index fb5d402105..d0eb63404b 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -2177,7 +2177,7 @@ static int mov_read_ctts(MOVContext *c, AVIOContext *pb, MOVAtom atom)
av_dlog(c->fc, "count=%d, duration=%d\n",
count, duration);
- if (FFABS(duration) > (1<<28) && i+2<entries) {
+ if (FFNABS(duration) < -(1<<28) && i+2<entries) {
av_log(c->fc, AV_LOG_WARNING, "CTTS invalid\n");
av_freep(&sc->ctts_data);
sc->ctts_count = 0;