diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2019-12-20 21:21:59 +0100 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-07-03 13:28:11 +0200 |
| commit | 445fee99b8a3b26c98d561a624b5271580518a1f (patch) | |
| tree | d96eb42a054869c4aa0e0f20503040d26df8939d | |
| parent | 3f165b3e63eadd312f97c76cd03d8745249278ec (diff) | |
| download | ffmpeg-445fee99b8a3b26c98d561a624b5271580518a1f.tar.gz | |
avformat/webmdashenc: Fix memleak upon realloc failure
The classical ptr = av_realloc(ptr, size).
Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
(cherry picked from commit 418e468699868a0265f8b439beedf64bb643b088)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
| -rw-r--r-- | libavformat/webmdashenc.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/libavformat/webmdashenc.c b/libavformat/webmdashenc.c index 9dc9b36d31..f3e23273ff 100644 --- a/libavformat/webmdashenc.c +++ b/libavformat/webmdashenc.c @@ -456,11 +456,12 @@ static int parse_adaptation_sets(AVFormatContext *s) state = parsing_streams; } else if (state == parsing_streams) { struct AdaptationSet *as = &w->as[w->nb_as - 1]; + int ret = av_reallocp_array(&as->streams, ++as->nb_streams, + sizeof(*as->streams)); + if (ret < 0) + return ret; q = p; while (*q != '\0' && *q != ',' && *q != ' ') q++; - as->streams = av_realloc(as->streams, sizeof(*as->streams) * ++as->nb_streams); - if (as->streams == NULL) - return AVERROR(ENOMEM); as->streams[as->nb_streams - 1] = to_integer(p, q - p + 1); if (as->streams[as->nb_streams - 1] < 0 || as->streams[as->nb_streams - 1] >= s->nb_streams) { |