]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
property: remove spurious incorrect comments
authorPauli <pauli@openssl.org>
Wed, 23 Jun 2021 04:18:25 +0000 (14:18 +1000)
committerPauli <pauli@openssl.org>
Thu, 24 Jun 2021 05:51:48 +0000 (15:51 +1000)
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/15871)

crypto/property/property.c

index c3f1c5ac587f46ca953cfaf0376f4139dd950e7d..a4cd612b9daa261e6a2acdb25f6a8628599c34e3 100644 (file)
@@ -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);