diff options
author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-01-27 09:28:19 +0100 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-07-02 10:15:06 +0200 |
commit | 38f968e380ca614f0be71ff5de8ab304c228b143 (patch) | |
tree | 039ff43251bdb7c4b6774f51af5430af4c9157f5 | |
parent | 89e3ecb725d01f8a215643197cd5806ed6a172d1 (diff) | |
download | ffmpeg-38f968e380ca614f0be71ff5de8ab304c228b143.tar.gz |
avformat/mov: Don't leak MOVFragmentStreamInfo on error
Fixes Coverity issue #1441933.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 43f58f2354bfab3819e44c1a97b0af75cc091226)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-rw-r--r-- | libavformat/mov.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 6ff5e0d82d..bad553d2fa 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1327,8 +1327,10 @@ static int update_frag_index(MOVContext *c, int64_t offset) for (i = 0; i < c->fc->nb_streams; i++) { // Avoid building frag index if streams lack track id. - if (c->fc->streams[i]->id < 0) + if (c->fc->streams[i]->id < 0) { + av_free(frag_stream_info); return AVERROR_INVALIDDATA; + } frag_stream_info[i].id = c->fc->streams[i]->id; frag_stream_info[i].sidx_pts = AV_NOPTS_VALUE; |