diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2025-06-21 23:15:17 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2025-06-26 00:40:49 +0200 |
| commit | 0ffe97d9b9e10e88db29d8d910c24ec00ef24edc (patch) | |
| tree | b7bfa70bc7a9c388ef139f10706d42b84634dd7d | |
| parent | 36c8eef42c3a5229c17aa4df2c9398b39b99b586 (diff) | |
| download | ffmpeg-0ffe97d9b9e10e88db29d8d910c24ec00ef24edc.tar.gz | |
avformat/mov: Check that sample_count is allocated in mov_parse_heif_items()
Fixes: NULL pointer dereference
Fixes: 416811958/clusterfuzz-testcase-minimized-ffmpeg_dem_MOV_fuzzer-5425269114732544
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Reviewed-by: James Almer <jamrial@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavformat/mov.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 0f4a5cd9a3..bd6b23466b 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -10341,7 +10341,7 @@ static int mov_parse_heif_items(AVFormatContext *s) st->codecpar->height = item->height; err = sanity_checks(s, sc, item->item_id); - if (err) + if (err || !sc->sample_count) return AVERROR_INVALIDDATA; sc->sample_sizes[0] = item->extent_length; |