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:28:39 +0100 |
| commit | c8a9fa6910c3cb6e9b5f8eb029eb6fc80dfc9cfe (patch) | |
| tree | 3f49acaee5fa5cb5a30e2669550c3f2ccd4bc04e /crypto/dsa | |
| parent | 202f7c56597eb6f57eba1ea31503a734e5fbf930 (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)
(cherry picked from commit ce1415ed2ce15305356cd028bcf7b9bc688d6d5c)
Diffstat (limited to 'crypto/dsa')
| -rw-r--r-- | crypto/dsa/dsa_gen.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/crypto/dsa/dsa_gen.c b/crypto/dsa/dsa_gen.c index 383d853b6d..30b20bbd88 100644 --- a/crypto/dsa/dsa_gen.c +++ b/crypto/dsa/dsa_gen.c @@ -292,8 +292,7 @@ int dsa_builtin_paramgen(DSA *ret, size_t bits, size_t qbits, if (seed_out) memcpy(seed_out, seed, qsize); } - if (ctx) - BN_CTX_end(ctx); + BN_CTX_end(ctx); BN_CTX_free(ctx); BN_MONT_CTX_free(mont); return ok; @@ -607,8 +606,7 @@ int dsa_builtin_paramgen2(DSA *ret, size_t L, size_t N, OPENSSL_free(seed); if (seed_out != seed_tmp) OPENSSL_free(seed_tmp); - if (ctx) - BN_CTX_end(ctx); + BN_CTX_end(ctx); BN_CTX_free(ctx); BN_MONT_CTX_free(mont); EVP_MD_CTX_free(mctx); |