[FFmpeg-cvslog] avformat/utils: Fix integer overflow of fps_first/last_dts

Michael Niedermayer git at videolan.org
Wed Mar 7 21:38:10 EET 2018


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Wed Mar  7 00:10:11 2018 +0100| [1b1362e408cd6acb63fef126b814b0d16562aa8e] | committer: Michael Niedermayer

avformat/utils: Fix integer overflow of fps_first/last_dts

Fixes: runtime error: signed integer overflow: 7738135736989908991 - -7898362169240453118 cannot be represented in type 'long'
Fixes: Chromium bug 796778
Reported-by: Matt Wolenetz <wolenetz at google.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1b1362e408cd6acb63fef126b814b0d16562aa8e
---

 libavformat/utils.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/utils.c b/libavformat/utils.c
index 3ca1ca2441..4d0b56c2a8 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -3778,7 +3778,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
             if (st->info->fps_last_dts != AV_NOPTS_VALUE &&
                 st->info->fps_last_dts_idx > st->info->fps_first_dts_idx &&
                 (pkt->dts - st->info->fps_last_dts) / 1000 >
-                (st->info->fps_last_dts     - st->info->fps_first_dts) /
+                (st->info->fps_last_dts     - (uint64_t)st->info->fps_first_dts) /
                 (st->info->fps_last_dts_idx - st->info->fps_first_dts_idx)) {
                 av_log(ic, AV_LOG_WARNING,
                        "DTS discontinuity in stream %d: packet %d with DTS "



More information about the ffmpeg-cvslog mailing list