From: Bernd Edlinger <bernd.edlinger@hotmail.de>
Date: Sun, 3 Dec 2023 10:29:52 +0000 (+0100)
Subject: Fix a possible memleak in apps/rehash.c
X-Git-Tag: openssl-3.3.0-alpha1~494
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=01709fcb8b609cfc47e277d20492c333bafb113e;p=thirdparty%2Fopenssl.git

Fix a possible memleak in apps/rehash.c

The OPENSSL_DIR_end was missing in case of error.

Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22920)
---

diff --git a/apps/rehash.c b/apps/rehash.c
index 4fea16fa38b..56a6ccea092 100644
--- a/apps/rehash.c
+++ b/apps/rehash.c
@@ -383,6 +383,7 @@ static int do_dir(const char *dirname, enum Hash h)
         if ((copy = OPENSSL_strdup(filename)) == NULL
                 || sk_OPENSSL_STRING_push(files, copy) == 0) {
             OPENSSL_free(copy);
+            OPENSSL_DIR_end(&d);
             BIO_puts(bio_err, "out of memory\n");
             errs = 1;
             goto err;