Browse Source

avformat/jacosubdec: 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 c13a752733)
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
release/3.2
Andreas Rheinhardt 5 years ago
parent
commit
01e64c3ba9
1 changed files with 4 additions and 2 deletions
  1. +4
    -2
      libavformat/jacosubdec.c

+ 4
- 2
libavformat/jacosubdec.c View File

@@ -187,8 +187,10 @@ static int jacosub_read_header(AVFormatContext *s)
AVPacket *sub;

sub = ff_subtitles_queue_insert(&jacosub->q, line, len, merge_line);
if (!sub)
return AVERROR(ENOMEM);
if (!sub) {
ret = AVERROR(ENOMEM);
goto fail;
}
sub->pos = pos;
merge_line = len > 1 && !strcmp(&line[len - 2], "\\\n");
continue;


Loading…
Cancel
Save