diff options
| author | Matt Caswell <matt@openssl.org> | 2021-11-08 16:30:43 +0000 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2021-11-12 17:16:14 +0000 |
| commit | 464c2b988ea149badabaf958a96fdc480df89dc7 (patch) | |
| tree | ce0df491c73a02b965968f7cb2dc43b00959914a /crypto/provider_child.c | |
| parent | 3b9de0c9aa791bd9e6f0534ec091accbdf15292f (diff) | |
| download | openssl-new-464c2b988ea149badabaf958a96fdc480df89dc7.tar.gz | |
Remove the isinited variable from child_prov_globals
This variable might have made sense at some point but it not longer does
so. It was being used to check whether we are still initing or not. If we
are still initing then the assumption was that we already hold the lock.
That assumption was untrue. We need to always take the lock.
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/16980)
Diffstat (limited to 'crypto/provider_child.c')
| -rw-r--r-- | crypto/provider_child.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/crypto/provider_child.c b/crypto/provider_child.c index 8f220c452f..1b92530301 100644 --- a/crypto/provider_child.c +++ b/crypto/provider_child.c @@ -22,7 +22,6 @@ DEFINE_STACK_OF(OSSL_PROVIDER) struct child_prov_globals { const OSSL_CORE_HANDLE *handle; const OSSL_CORE_HANDLE *curr_prov; - unsigned int isinited:1; CRYPTO_RWLOCK *lock; OSSL_FUNC_core_get_libctx_fn *c_get_libctx; OSSL_FUNC_provider_register_child_cb_fn *c_provider_register_child_cb; @@ -110,11 +109,7 @@ static int provider_create_child_cb(const OSSL_CORE_HANDLE *prov, void *cbdata) if (gbl == NULL) return 0; - /* - * If !gbl->isinited, then we are still initing and we already hold the - * lock - so don't take it again. - */ - if (gbl->isinited && !CRYPTO_THREAD_write_lock(gbl->lock)) + if (!CRYPTO_THREAD_write_lock(gbl->lock)) return 0; provname = gbl->c_prov_name(prov); @@ -161,8 +156,7 @@ static int provider_create_child_cb(const OSSL_CORE_HANDLE *prov, void *cbdata) ret = 1; err: - if (gbl->isinited) - CRYPTO_THREAD_unlock(gbl->lock); + CRYPTO_THREAD_unlock(gbl->lock); return ret; } @@ -272,8 +266,6 @@ int ossl_provider_init_as_child(OSSL_LIB_CTX *ctx, ctx)) return 0; - gbl->isinited = 1; - return 1; } |