summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Rheinhardt <[email protected]>2020-06-14 03:29:17 +0200
committerAndreas Rheinhardt <[email protected]>2020-07-02 01:01:31 +0200
commit1f09053c79aa65eb6729730ff5309b26f87e79a7 (patch)
tree4c201afc0d8b2f1f88d677c1858435210c187209
parentdb822733687c270a37200f25e7c4c8aa53438435 (diff)
avformat/webvttdec: Fix memleak upon read header failure
The already parsed subtitles (contained in an FFDemuxSubtitlesQueue) would leak if an error happened upon reading a subsequent subtitle. Signed-off-by: Andreas Rheinhardt <[email protected]> (cherry picked from commit c784fe8b867e42a1c8d2c48d7046e3e0cce7ec31) Signed-off-by: Andreas Rheinhardt <[email protected]>
-rw-r--r--libavformat/webvttdec.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c
index 0aeb8a63f4..8e9c562683 100644
--- a/libavformat/webvttdec.c
+++ b/libavformat/webvttdec.c
@@ -165,6 +165,8 @@ static int webvtt_read_header(AVFormatContext *s)
ff_subtitles_queue_finalize(s, &webvtt->q);
end:
+ if (res < 0)
+ ff_subtitles_queue_clean(&webvtt->q);
av_bprint_finalize(&cue, NULL);
av_bprint_finalize(&header, NULL);
return res;