[FFmpeg-cvslog] avformat/asfdec_o: Check offset before adding index entry

Michael Niedermayer git at videolan.org
Fri Oct 21 00:28:05 EEST 2022


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Wed Oct  5 21:46:42 2022 +0200| [557aa7772e6cd646baa7bfd0046ac7e37442cff3] | committer: Michael Niedermayer

avformat/asfdec_o: Check offset before adding index entry

Fixes: signed integer overflow: 9223372036854550860 + 530259564 cannot be represented in type 'long'
Fixes: 49093/clusterfuzz-testcase-minimized-ffmpeg_dem_ASF_O_fuzzer-4697179192688640

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 libavformat/asfdec_o.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavformat/asfdec_o.c b/libavformat/asfdec_o.c
index e837ca62e7..2b407c016f 100644
--- a/libavformat/asfdec_o.c
+++ b/libavformat/asfdec_o.c
@@ -888,6 +888,8 @@ static int asf_read_simple_index(AVFormatContext *s, const GUIDParseTable *g)
             av_log(s, AV_LOG_ERROR, "Skipping failed in asf_read_simple_index.\n");
             return offset;
         }
+        if (asf->first_packet_offset > INT64_MAX - asf->packet_size * pkt_num)
+            return AVERROR_INVALIDDATA;
         if (prev_pkt_num != pkt_num) {
             av_add_index_entry(st, asf->first_packet_offset + asf->packet_size *
                                pkt_num, av_rescale(interval, i, 10000),



More information about the ffmpeg-cvslog mailing list