diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2021-02-22 20:44:36 +0100 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2021-03-14 23:29:51 +0100 |
commit | 1c88098651469bf749eedd51f3227ec7edb4c2de (patch) | |
tree | 8eb13fbe55ebadfcf845f766fc9d386117b3e856 | |
parent | cd3ab3a09b7ea19ffef559efd4aae6292e196b42 (diff) | |
download | ffmpeg-1c88098651469bf749eedd51f3227ec7edb4c2de.tar.gz |
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>
-rw-r--r-- | libavformat/samidec.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libavformat/samidec.c b/libavformat/samidec.c index cf5076c7b8..d84f56ce6f 100644 --- a/libavformat/samidec.c +++ b/libavformat/samidec.c @@ -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; } |