Browse Source

avformat/samidec: Deallocate hdr_buf

Fixes: memleak
Fixes: 30841/clusterfuzz-testcase-minimized-ffmpeg_dem_SAMI_fuzzer-4521799196999680

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
tags/n4.4
Michael Niedermayer 5 years ago
parent
commit
1c88098651
1 changed files with 2 additions and 0 deletions
  1. +2
    -0
      libavformat/samidec.c

+ 2
- 0
libavformat/samidec.c View File

@@ -89,6 +89,7 @@ static int sami_read_header(AVFormatContext *s)
sub = ff_subtitles_queue_insert(&sami->q, buf.str, buf.len, !is_sync);
if (!sub) {
res = AVERROR(ENOMEM);
av_bprint_finalize(&hdr_buf, NULL);
goto end;
}
if (is_sync) {
@@ -97,6 +98,7 @@ static int sami_read_header(AVFormatContext *s)
sub->pts = p ? strtol(p, NULL, 10) : 0;
if (sub->pts <= INT64_MIN/2 || sub->pts >= INT64_MAX/2) {
res = AVERROR_PATCHWELCOME;
av_bprint_finalize(&hdr_buf, NULL);
goto end;
}



Loading…
Cancel
Save