[FFmpeg-cvslog] avformat/tedcaptionsdec: Fix memleak upon read header failure
Andreas Rheinhardt
git at videolan.org
Fri Jul 3 16:09:35 EEST 2020
ffmpeg | branch: release/3.3 | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Sun Jun 14 03:27:43 2020 +0200| [d8c5124ad987ea9af08b91ebf954bcdf16bb98df] | committer: Andreas Rheinhardt
avformat/tedcaptionsdec: Fix memleak upon read header failure
The already parsed subtitles (contained in an FFDemuxSubtitlesQueue)
would leak if allocating the AVStream for the subtitles fails.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
(cherry picked from commit 337783b118d4cc265759c103b672dd5d5d3e7cb8)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d8c5124ad987ea9af08b91ebf954bcdf16bb98df
---
libavformat/tedcaptionsdec.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/libavformat/tedcaptionsdec.c b/libavformat/tedcaptionsdec.c
index 774d4993b5..21d44c17f4 100644
--- a/libavformat/tedcaptionsdec.c
+++ b/libavformat/tedcaptionsdec.c
@@ -275,10 +275,13 @@ static int parse_file(AVIOContext *pb, FFDemuxSubtitlesQueue *subs)
static av_cold int tedcaptions_read_header(AVFormatContext *avf)
{
TEDCaptionsDemuxer *tc = avf->priv_data;
- AVStream *st;
+ AVStream *st = avformat_new_stream(avf, NULL);
int ret, i;
AVPacket *last;
+ if (!st)
+ return AVERROR(ENOMEM);
+
ret = parse_file(avf->pb, &tc->subs);
if (ret < 0) {
if (ret == AVERROR_INVALIDDATA)
@@ -292,9 +295,6 @@ static av_cold int tedcaptions_read_header(AVFormatContext *avf)
tc->subs.subs[i].pts += tc->start_time;
last = &tc->subs.subs[tc->subs.nb_subs - 1];
- st = avformat_new_stream(avf, NULL);
- if (!st)
- return AVERROR(ENOMEM);
st->codecpar->codec_type = AVMEDIA_TYPE_SUBTITLE;
st->codecpar->codec_id = AV_CODEC_ID_TEXT;
avpriv_set_pts_info(st, 64, 1, 1000);
More information about the ffmpeg-cvslog
mailing list