summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPauli <ppzgs1@gmail.com>2021-03-22 08:47:58 +1000
committerPauli <pauli@openssl.org>2021-03-24 09:40:26 +1000
commit9d8c53ed164f325a28bee657b09cf4b30d62a08f (patch)
treeeee207a590fd277517566d07597e7cb74bd3a749
parent9ca269af63a5772d3e9c28c4e4893fafb306202e (diff)
dh: fix coverty 1474423: resource leak
Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14637)
-rw-r--r--crypto/dh/dh_backend.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/dh/dh_backend.c b/crypto/dh/dh_backend.c
index 8da830f9d8..97f5271a5a 100644
--- a/crypto/dh/dh_backend.c
+++ b/crypto/dh/dh_backend.c
@@ -161,6 +161,7 @@ DH *ossl_dh_key_from_pkcs8(const PKCS8_PRIV_KEY_INFO *p8inf,
if ((privkey_bn = BN_secure_new()) == NULL
|| !ASN1_INTEGER_to_BN(privkey, privkey_bn)) {
ERR_raise(ERR_LIB_DH, DH_R_BN_ERROR);
+ BN_clear_free(privkey_bn);
goto dherr;
}
if (!DH_set0_key(dh, NULL, privkey_bn))