diff options
| -rw-r--r-- | crypto/property/property.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/crypto/property/property.c b/crypto/property/property.c index c3f1c5ac58..a4cd612b9d 100644 --- a/crypto/property/property.c +++ b/crypto/property/property.c @@ -273,12 +273,7 @@ int ossl_method_store_add(OSSL_METHOD_STORE *store, const OSSL_PROVIDER *prov, } impl->provider = prov; - /* - * Insert into the hash table if required. - * - * A write lock is used unconditionally because we wend our way down to the - * property string code which isn't locking friendly. - */ + /* Insert into the hash table if required */ if (!ossl_property_write_lock(store)) { OPENSSL_free(impl); return 0; @@ -418,10 +413,7 @@ int ossl_method_store_fetch(OSSL_METHOD_STORE *store, int nid, if (nid <= 0 || method == NULL || store == NULL) return 0; - /* - * This only needs to be a read lock, because queries never create property - * names or value and thus don't modify any of the property string layer. - */ + /* This only needs to be a read lock, because the query won't create anything */ if (!ossl_property_read_lock(store)) return 0; alg = ossl_method_store_retrieve(store, nid); |