summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoryangyangtiantianlonglong <yangtianlong1224@163.com>2022-10-27 17:17:55 +0800
committerTomas Mraz <tomas@openssl.org>2022-11-02 10:58:20 +0100
commit15c8df81083f31dd35aedbe2d58ec702d0c0dc65 (patch)
treea82e810c82246352f423903934e036a9ebd0b629
parent41e4f72d4cead8caf65f046aea706fe307be1c1e (diff)
Potential null pointer reference
Reviewed-by: Shane Lontis <shane.lontis@oracle.com> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/19429)
-rw-r--r--test/ectest.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/test/ectest.c b/test/ectest.c
index 8e62cf01e6..c0337e1c95 100644
--- a/test/ectest.c
+++ b/test/ectest.c
@@ -2724,6 +2724,13 @@ static int custom_params_test(int id)
if (!TEST_ptr(ctx = BN_CTX_new()))
return 0;
+ BN_CTX_start(ctx);
+ if (!TEST_ptr(p = BN_CTX_get(ctx))
+ || !TEST_ptr(a = BN_CTX_get(ctx))
+ || !TEST_ptr(b = BN_CTX_get(ctx))
+ || !TEST_ptr(k = BN_CTX_get(ctx)))
+ goto err;
+
if (!TEST_ptr(group = EC_GROUP_new_by_curve_name(nid)))
goto err;
@@ -2735,13 +2742,6 @@ static int custom_params_test(int id)
}
#endif
- BN_CTX_start(ctx);
- if (!TEST_ptr(p = BN_CTX_get(ctx))
- || !TEST_ptr(a = BN_CTX_get(ctx))
- || !TEST_ptr(b = BN_CTX_get(ctx))
- || !TEST_ptr(k = BN_CTX_get(ctx)))
- goto err;
-
/* expected byte length of encoded points */
bsize = (EC_GROUP_get_degree(group) + 7) / 8;
bsize = 1 + 2 * bsize; /* UNCOMPRESSED_POINT format */