[FFmpeg-devel] [PATCH 1/3] avformat/mov: Check sample_sizes before using it

Michael Niedermayer michael at niedermayer.cc
Sat Jul 27 00:08:30 EEST 2024


Fixes: NULL pointer dereference
Fixes: 70569/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-5247918563459072

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 libavformat/mov.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index b74e43e2140..63db7d59a58 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -10060,6 +10060,10 @@ static int mov_read_header(AVFormatContext *s)
 
             st = item->st;
             sc = st->priv_data;
+
+            if (!sc->sample_sizes || !sc->sample_count)
+                return AVERROR_INVALIDDATA;
+
             st->codecpar->width  = item->width;
             st->codecpar->height = item->height;
 
-- 
2.45.2



More information about the ffmpeg-devel mailing list