From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Fri, 25 Jan 2019 07:51:47 +0000 (+0700)
Subject: Correct function here is HMAC_CTX_cleanup() to free any dynamically allocated buffers
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a160a422ecb13569af770229fc079185a92a9fbe;p=thirdparty%2Ffreeradius-server.git

Correct function here is HMAC_CTX_cleanup() to free any dynamically allocated buffers
---

diff --git a/src/lib/tls/missing.h b/src/lib/tls/missing.h
index 8c79e03b752..8488bfc66da 100644
--- a/src/lib/tls/missing.h
+++ b/src/lib/tls/missing.h
@@ -35,9 +35,7 @@ static inline HMAC_CTX *HMAC_CTX_new(void)
 
 static inline void HMAC_CTX_free(HMAC_CTX *ctx)
 {
-        if (ctx == NULL) {
-                return;
-        }
+        if (ctx == NULL) return;
         HMAC_CTX_cleanup(ctx);
         OPENSSL_free(ctx);
 }
@@ -46,8 +44,7 @@ static inline int HMAC_CTX_reset(HMAC_CTX *ctx)
 {
 	if (!ctx) return 0;
 
-	memset(ctx, 0, sizeof(*ctx));
-        HMAC_CTX_init(ctx);
+        HMAC_CTX_cleanup(ctx);
 
 	return 1;
 }