aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-06-14 03:12:00 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-07-03 00:02:13 +0200
commit9b021e24f723a707c2bd90ca557cec8ac454386c (patch)
treeeabb1da5773ac99502dd784ff642edf0385a1f4b
parentbde71f0c9b6fdb218f9910e180e3ac362d6ac9ba (diff)
downloadffmpeg-9b021e24f723a707c2bd90ca557cec8ac454386c.tar.gz
avformat/lrcdec: 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 <andreas.rheinhardt@gmail.com> (cherry picked from commit d38694cea9f289b3f9dcce1a2f07746d029b35f3) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-rw-r--r--libavformat/lrcdec.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/libavformat/lrcdec.c b/libavformat/lrcdec.c
index f4e9a4efa9..45f0bf24a7 100644
--- a/libavformat/lrcdec.c
+++ b/libavformat/lrcdec.c
@@ -202,6 +202,7 @@ static int lrc_read_header(AVFormatContext *s)
sub = ff_subtitles_queue_insert(&lrc->q, line.str + ts_strlength,
line.len - ts_strlength, 0);
if(!sub) {
+ ff_subtitles_queue_clean(&lrc->q);
return AVERROR(ENOMEM);
}
sub->pos = pos;