From d24c6f756908de13f9efd9dc765946fd0a79fbc5 Mon Sep 17 00:00:00 2001 From: Micah Snyder Date: Fri, 7 Apr 2023 10:43:09 -0700 Subject: [PATCH] Coverity-401434: Switch to realloc that does not free after failure Because we free after the done-label. --- libclamav/unzip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libclamav/unzip.c b/libclamav/unzip.c index 1f063d7b7..0316b0858 100644 --- a/libclamav/unzip.c +++ b/libclamav/unzip.c @@ -1061,7 +1061,7 @@ cl_error_t index_the_central_directory( goto done; } - zip_catalogue_new = cli_realloc2(zip_catalogue, sizeof(struct zip_record) * ZIP_RECORDS_CHECK_BLOCKSIZE * (num_record_blocks + 1)); + zip_catalogue_new = cli_realloc(zip_catalogue, sizeof(struct zip_record) * ZIP_RECORDS_CHECK_BLOCKSIZE * (num_record_blocks + 1)); if (NULL == zip_catalogue_new) { status = CL_EMEM; goto done;