From 99b7beafd212144fc3c77a6c09fc48f06f245c29 Mon Sep 17 00:00:00 2001
From: Pauli <pauli@openssl.org>
Date: Thu, 22 Jun 2023 09:33:40 +1000
Subject: ecx: update to structure based atomics

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21260)
---
 crypto/ec/ecx_backend.c | 11 ++++-------
 crypto/ec/ecx_key.c     | 18 +++++++-----------
 2 files changed, 11 insertions(+), 18 deletions(-)

(limited to 'crypto')

diff --git a/crypto/ec/ecx_backend.c b/crypto/ec/ecx_backend.c
index 7a43274d51..0107a612a8 100644
--- a/crypto/ec/ecx_backend.c
+++ b/crypto/ec/ecx_backend.c
@@ -113,17 +113,13 @@ ECX_KEY *ossl_ecx_key_dup(const ECX_KEY *key, int selection)
     if (ret == NULL)
         return NULL;
 
-    ret->lock = CRYPTO_THREAD_lock_new();
-    if (ret->lock == NULL) {
-        OPENSSL_free(ret);
-        return NULL;
-    }
-
     ret->libctx = key->libctx;
     ret->haspubkey = key->haspubkey;
     ret->keylen = key->keylen;
     ret->type = key->type;
-    ret->references = 1;
+
+    if (!CRYPTO_NEW_REF(&ret->references, 1))
+        goto err;
 
     if (key->propq != NULL) {
         ret->propq = OPENSSL_strdup(key->propq);
@@ -146,6 +142,7 @@ ECX_KEY *ossl_ecx_key_dup(const ECX_KEY *key, int selection)
     return ret;
 
 err:
+    CRYPTO_FREE_REF(&ret->references);
     ossl_ecx_key_free(ret);
     return NULL;
 }
diff --git a/crypto/ec/ecx_key.c b/crypto/ec/ecx_key.c
index 548e49091d..36276ce98e 100644
--- a/crypto/ec/ecx_key.c
+++ b/crypto/ec/ecx_key.c
@@ -42,24 +42,20 @@ ECX_KEY *ossl_ecx_key_new(OSSL_LIB_CTX *libctx, ECX_KEY_TYPE type, int haspubkey
         break;
     }
     ret->type = type;
-    ret->references = 1;
+
+    if (!CRYPTO_NEW_REF(&ret->references, 1))
+        goto err;
 
     if (propq != NULL) {
         ret->propq = OPENSSL_strdup(propq);
         if (ret->propq == NULL)
             goto err;
     }
-
-    ret->lock = CRYPTO_THREAD_lock_new();
-    if (ret->lock == NULL) {
-        ERR_raise(ERR_LIB_EC, ERR_R_CRYPTO_LIB);
-        goto err;
-    }
     return ret;
 err:
     if (ret != NULL) {
         OPENSSL_free(ret->propq);
-        CRYPTO_THREAD_lock_free(ret->lock);
+        CRYPTO_FREE_REF(&ret->references);
     }
     OPENSSL_free(ret);
     return NULL;
@@ -72,7 +68,7 @@ void ossl_ecx_key_free(ECX_KEY *key)
     if (key == NULL)
         return;
 
-    CRYPTO_DOWN_REF(&key->references, &i, key->lock);
+    CRYPTO_DOWN_REF(&key->references, &i);
     REF_PRINT_COUNT("ECX_KEY", key);
     if (i > 0)
         return;
@@ -80,7 +76,7 @@ void ossl_ecx_key_free(ECX_KEY *key)
 
     OPENSSL_free(key->propq);
     OPENSSL_secure_clear_free(key->privkey, key->keylen);
-    CRYPTO_THREAD_lock_free(key->lock);
+    CRYPTO_FREE_REF(&key->references);
     OPENSSL_free(key);
 }
 
@@ -93,7 +89,7 @@ int ossl_ecx_key_up_ref(ECX_KEY *key)
 {
     int i;
 
-    if (CRYPTO_UP_REF(&key->references, &i, key->lock) <= 0)
+    if (CRYPTO_UP_REF(&key->references, &i) <= 0)
         return 0;
 
     REF_PRINT_COUNT("ECX_KEY", key);
-- 
cgit v1.2.3