[FFmpeg-cvslog] avformat/mov: fix overflow in corrected_dts calculation

James Almer git at videolan.org
Mon Mar 3 23:16:49 EET 2025


ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Sat Mar  1 23:51:25 2025 -0300| [25c439296ba1e835599370de331f9a089f7efe37] | committer: James Almer

avformat/mov: fix overflow in corrected_dts calculation

Fixes: Integer-overflow
Fixes: 400093647/clusterfuzz-testcase-minimized-media_metadata_parser_fuzzer-4794341562187776

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: James Almer <jamrial at gmail.com>

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

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

diff --git a/libavformat/mov.c b/libavformat/mov.c
index ee44b10435..57d9364ea2 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -3539,7 +3539,7 @@ static int mov_read_stts(MOVContext *c, AVIOContext *pb, MOVAtom atom)
             av_log(c->fc, AV_LOG_WARNING, "Too large sample offset %u in stts entry %u with count %u in st:%d. Clipping to 1.\n",
                    sample_duration, i, sample_count, st->index);
             sc->stts_data[i].duration = 1;
-            corrected_dts += (delta_magnitude < 0 ? (int64_t)delta_magnitude : 1) * sample_count;
+            corrected_dts = av_sat_add64(corrected_dts, (delta_magnitude < 0 ? (int64_t)delta_magnitude : 1) * sample_count);
         } else {
             corrected_dts += sample_duration * (uint64_t)sample_count;
         }



More information about the ffmpeg-cvslog mailing list