From 7cfc42151e235392562fcfb025b8da03287b5bda Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sun, 7 Jul 2019 22:50:42 +0200 Subject: [PATCH] tools/target_dec_fuzzer: Free parser in case of avcodec_open2() failure Fixes: memleak Fixes: part of 15529/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_LIBVPX_VP8_fuzzer-5140143700180992 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer (cherry picked from commit 508ce5839e0bf78ce8813eb1b38cce0d416a408e) Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index a94d5e6a92..d8b1a049ff 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target_dec_fuzzer.c @@ -191,6 +191,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) { if (res < 0) { av_free(ctx); av_free(parser_avctx); + av_parser_close(parser); return 0; // Failure of avcodec_open2() does not imply that a issue was found } parser_avctx->codec_id = ctx->codec_id;