diff options
author | Matt Caswell <matt@openssl.org> | 2021-11-09 14:32:14 +0000 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2021-11-12 17:16:14 +0000 |
commit | 4aced11785f2e54875ad56f30c05bdee02b6e4e2 (patch) | |
tree | bc6dc99406a71e8c5233d8950e6fc58d5e4df4f2 /crypto/provider_core.c | |
parent | 1e8ed3e596162d7490b26fb12e58af5208f52402 (diff) |
Use a write lock during ossl_provider_find()
A "find" operation on a stack can end up sorting the underlying stack. In
this case it is necessary to use a "write" lock to synchronise access to
the stack across multiple threads.
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/16980)
Diffstat (limited to 'crypto/provider_core.c')
-rw-r--r-- | crypto/provider_core.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/crypto/provider_core.c b/crypto/provider_core.c index c9dba6dd86..a46a96cc93 100644 --- a/crypto/provider_core.c +++ b/crypto/provider_core.c @@ -423,7 +423,11 @@ OSSL_PROVIDER *ossl_provider_find(OSSL_LIB_CTX *libctx, const char *name, #endif tmpl.name = (char *)name; - if (!CRYPTO_THREAD_read_lock(store->lock)) + /* + * A "find" operation can sort the stack, and therefore a write lock is + * required. + */ + if (!CRYPTO_THREAD_write_lock(store->lock)) return NULL; if ((i = sk_OSSL_PROVIDER_find(store->providers, &tmpl)) != -1) prov = sk_OSSL_PROVIDER_value(store->providers, i); |