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/dh/dh_check.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/dh/dh_check.c')
| -rw-r--r-- | crypto/dh/dh_check.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/crypto/dh/dh_check.c b/crypto/dh/dh_check.c index 31332cd1f3..cd7f70b672 100644 --- a/crypto/dh/dh_check.c +++ b/crypto/dh/dh_check.c @@ -58,10 +58,8 @@ int DH_check_params(const DH *dh, int *ret) ok = 1; err: - if (ctx != NULL) { - BN_CTX_end(ctx); - BN_CTX_free(ctx); - } + BN_CTX_end(ctx); + BN_CTX_free(ctx); return ok; } @@ -171,10 +169,8 @@ int DH_check(const DH *dh, int *ret) } ok = 1; err: - if (ctx != NULL) { - BN_CTX_end(ctx); - BN_CTX_free(ctx); - } + BN_CTX_end(ctx); + BN_CTX_free(ctx); return ok; } @@ -225,9 +221,7 @@ int DH_check_pub_key(const DH *dh, const BIGNUM *pub_key, int *ret) ok = 1; err: - if (ctx != NULL) { - BN_CTX_end(ctx); - BN_CTX_free(ctx); - } + BN_CTX_end(ctx); + BN_CTX_free(ctx); return ok; } |