From: Aaron Thompson <dev@aaront.org>
Date: Tue, 31 Mar 2020 07:19:16 +0000 (+0000)
Subject: Remove an unnecessary call to BN_CTX_free.
X-Git-Tag: openssl-3.0.0-alpha1~164
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=94468c775c8e23dae8549ca15b4f9e0718650b0c;p=oweals%2Fopenssl.git

Remove an unnecessary call to BN_CTX_free.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/11452)
---

diff --git a/crypto/ec/ec_check.c b/crypto/ec/ec_check.c
index 1283d8404f..a4cc8073de 100644
--- a/crypto/ec/ec_check.c
+++ b/crypto/ec/ec_check.c
@@ -19,12 +19,12 @@
 int EC_GROUP_check_named_curve(const EC_GROUP *group, int nist_only,
                                BN_CTX *ctx)
 {
-    int nid = NID_undef;
+    int nid;
     BN_CTX *new_ctx = NULL;
 
     if (group == NULL) {
         ECerr(0, ERR_R_PASSED_NULL_PARAMETER);
-        goto err;
+        return NID_undef;
     }
 
     if (ctx == NULL) {
@@ -39,7 +39,6 @@ int EC_GROUP_check_named_curve(const EC_GROUP *group, int nist_only,
     if (nid > 0 && nist_only && EC_curve_nid2nist(nid) == NULL)
         nid = NID_undef;
 
-err:
     BN_CTX_free(new_ctx);
     return nid;
 }