From 460a5c12b47359d30a874dd6fcff11a3cb4c2f00 Mon Sep 17 00:00:00 2001 From: Emanuele Torre Date: Tue, 28 Nov 2023 20:41:09 +0100 Subject: [PATCH] jq_fuzz_parse_extended.c: don't jv_free() twice jv_dump() frees its argument. I missed this problem before merging #2952, whoops! =) fixup from eb3b5654bbd285fa70bab8ca71f2284354adf625 --- tests/jq_fuzz_parse_extended.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/jq_fuzz_parse_extended.c b/tests/jq_fuzz_parse_extended.c index 7f212fce..aa49544a 100644 --- a/tests/jq_fuzz_parse_extended.c +++ b/tests/jq_fuzz_parse_extended.c @@ -25,9 +25,10 @@ int LLVMFuzzerTestOneInput(uint8_t *data, size_t size) { jv res = jv_parse_custom_flags(null_terminated, fuzz_flags); if (jv_is_valid(res)) { jv_dump(res, dump_flags); + } else { + jv_free(res); } - jv_free(res); - + // Free the null-terminated string free(null_terminated);