diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2018-05-15 17:06:59 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2018-06-15 22:31:13 +0200 |
| commit | 1d01a3b34c924998da310c4776af36047d1b7fde (patch) | |
| tree | cff6b4d3a6b35417356735f0efd926239bee8fef /libavformat | |
| parent | 2742cb10c7337683967f494ba8090d87eb0af7d7 (diff) | |
| download | ffmpeg-1d01a3b34c924998da310c4776af36047d1b7fde.tar.gz | |
avformat/mov: Break out early if chunk_count is 0 in mov_build_index()
Without this some operations might overflow (undefined behavior)
even though the index adding loop would never execute
No testcase known
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 56e76bd0579cc7f7b28860885d9e569a39daf41b)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavformat')
| -rw-r--r-- | libavformat/mov.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 0667ba3ded..da0b1942b9 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -3899,6 +3899,9 @@ static void mov_build_index(MOVContext *mov, AVStream *st) } else { unsigned chunk_samples, total = 0; + if (!sc->chunk_count) + return; + // compute total chunk count for (i = 0; i < sc->stsc_count; i++) { unsigned count, chunk_count; |