commit a0238b7ed87998c48b1c92bad7fa82dcbba507f9
author Pauli <pauli@openssl.org> Wed Mar 16 13:48:27 2022 +1100
committer Pauli <pauli@openssl.org> Wed Mar 23 11:10:32 2022 +1100
tree 56a8839a1af65c2d5f276bd1612cb6e0e2b20471
parent 6889ebff01fa8cd7e5905f3f242edfed55fca443

Fix Coverity 1201763 uninitialised pointer read

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

crypto/bn/bn_exp.c

diff --git a/crypto/bn/bn_exp.c b/crypto/bn/bn_exp.c
index bb20e16..9b6f18a 100644
--- a/crypto/bn/bn_exp.c
+++ b/crypto/bn/bn_exp.c
@@ -187,13 +187,14 @@
         return ret;
     }
 
+    BN_RECP_CTX_init(&recp);
+
     BN_CTX_start(ctx);
     aa = BN_CTX_get(ctx);
     val[0] = BN_CTX_get(ctx);
     if (val[0] == NULL)
         goto err;
 
-    BN_RECP_CTX_init(&recp);
     if (m->neg) {
         /* ignore sign of 'm' */
         if (!BN_copy(aa, m))