diff options
author | Shane Lontis <shane.lontis@oracle.com> | 2019-03-19 09:58:09 +1000 |
---|---|---|
committer | Richard Levitte <levitte@openssl.org> | 2019-03-19 07:25:48 +0100 |
commit | ce1415ed2ce15305356cd028bcf7b9bc688d6d5c (patch) | |
tree | e9f4d5ed4bbb781d42b09c6a2c9bebd73fe8917f /crypto/ec/ecp_smpl.c | |
parent | 226f2bf191ba8c2b33749ddc80c9ace051bebf80 (diff) |
Added NULL check to BN_clear() & BN_CTX_end()
Reviewed-by: Paul Dale <paul.dale@oracle.com>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/8518)
Diffstat (limited to 'crypto/ec/ecp_smpl.c')
-rw-r--r-- | crypto/ec/ecp_smpl.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/crypto/ec/ecp_smpl.c b/crypto/ec/ecp_smpl.c index f80f98a169..2fd2ce46ea 100644 --- a/crypto/ec/ecp_smpl.c +++ b/crypto/ec/ecp_smpl.c @@ -307,8 +307,7 @@ int ec_GFp_simple_group_check_discriminant(const EC_GROUP *group, BN_CTX *ctx) ret = 1; err: - if (ctx != NULL) - BN_CTX_end(ctx); + BN_CTX_end(ctx); BN_CTX_free(new_ctx); return ret; } @@ -787,8 +786,7 @@ int ec_GFp_simple_add(const EC_GROUP *group, EC_POINT *r, const EC_POINT *a, ret = 1; end: - if (ctx) /* otherwise we already called BN_CTX_end */ - BN_CTX_end(ctx); + BN_CTX_end(ctx); BN_CTX_free(new_ctx); return ret; } |