diff options
author | FdaSilvaYY <fdasilvayy@gmail.com> | 2016-03-08 20:11:48 +0100 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2016-04-28 14:37:41 -0400 |
commit | 2bbf0baa9565cce937d85592f12d1279871dfaf6 (patch) | |
tree | 43fd8d6d74ea714fbef4e5c4d1aeea41c14572bb /crypto/dh | |
parent | 25a807bcb9509c6fccfbbd9d02119772ccb0f23e (diff) |
Add checks on CRYPTO_new_ex_data return value...
with some adaptation to new multi-threading API.
Once reference, lock, meth and flag fields are setup,
DSA_free/DH_free can be called directly.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/996)
Diffstat (limited to 'crypto/dh')
-rw-r--r-- | crypto/dh/dh_lib.c | 31 |
1 files changed, 14 insertions, 17 deletions
diff --git a/crypto/dh/dh_lib.c b/crypto/dh/dh_lib.c index 9db45763fa..650ca43da0 100644 --- a/crypto/dh/dh_lib.c +++ b/crypto/dh/dh_lib.c @@ -109,13 +109,20 @@ DH *DH_new_method(ENGINE *engine) return NULL; } + ret->references = 1; + ret->lock = CRYPTO_THREAD_lock_new(); + if (ret->lock == NULL) { + OPENSSL_free(ret); + return NULL; + } + ret->meth = DH_get_default_method(); #ifndef OPENSSL_NO_ENGINE + ret->flags = ret->meth->flags; /* early default init */ if (engine) { if (!ENGINE_init(engine)) { DHerr(DH_F_DH_NEW_METHOD, ERR_R_ENGINE_LIB); - OPENSSL_free(ret); - return NULL; + goto err; } ret->engine = engine; } else @@ -124,29 +131,19 @@ DH *DH_new_method(ENGINE *engine) ret->meth = ENGINE_get_DH(ret->engine); if (ret->meth == NULL) { DHerr(DH_F_DH_NEW_METHOD, ERR_R_ENGINE_LIB); - ENGINE_finish(ret->engine); - OPENSSL_free(ret); - return NULL; + goto err; } } #endif - ret->references = 1; ret->flags = ret->meth->flags; - CRYPTO_new_ex_data(CRYPTO_EX_INDEX_DH, ret, &ret->ex_data); - - ret->lock = CRYPTO_THREAD_lock_new(); - if (ret->lock == NULL) { -#ifndef OPENSSL_NO_ENGINE - ENGINE_finish(ret->engine); -#endif - CRYPTO_free_ex_data(CRYPTO_EX_INDEX_DH, ret, &ret->ex_data); - OPENSSL_free(ret); - return NULL; - } + if (!CRYPTO_new_ex_data(CRYPTO_EX_INDEX_DH, ret, &ret->ex_data)) + goto err; if ((ret->meth->init != NULL) && !ret->meth->init(ret)) { + DHerr(DH_F_DH_NEW_METHOD, ERR_R_INIT_FAIL); +err: DH_free(ret); ret = NULL; } |