From: Neil Horman <nhorman@openssl.org>
Date: Wed, 18 Jun 2025 11:33:47 +0000 (-0400)
Subject: Restore use of crypto_thread_default_context
X-Git-Tag: openssl-3.6.0-alpha1~548
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5466197f16f61e08af70c73e622534f2b984c419;p=thirdparty%2Fopenssl.git

Restore use of crypto_thread_default_context

This thread-local key isn't per thread*context, and is potentially in a
hot path for users of openssl that switch their default contexts
regularly, so lets leave it as an OS level key, rather than a library
managed key.

Reviewed-by: Saša Nedvědický <sashan@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/27794)
---

diff --git a/crypto/context.c b/crypto/context.c
index 32f80cf5220..80a87562cd8 100644
--- a/crypto/context.c
+++ b/crypto/context.c
@@ -378,15 +378,24 @@ static int context_deinit(OSSL_LIB_CTX *ctx)
 static OSSL_LIB_CTX default_context_int;
 
 static CRYPTO_ONCE default_context_init = CRYPTO_ONCE_STATIC_INIT;
+static CRYPTO_THREAD_LOCAL default_context_thread_local;
 static int default_context_inited = 0;
 
 DEFINE_RUN_ONCE_STATIC(default_context_do_init)
 {
+    if (!CRYPTO_THREAD_init_local(&default_context_thread_local, NULL))
+        goto err;
+
     if (!context_init(&default_context_int))
-        return 0;
+        goto deinit_thread;
 
     default_context_inited = 1;
     return 1;
+
+deinit_thread:
+    CRYPTO_THREAD_cleanup_local(&default_context_thread_local);
+err:
+    return 0;
 }
 
 void ossl_lib_ctx_default_deinit(void)
@@ -394,6 +403,7 @@ void ossl_lib_ctx_default_deinit(void)
     if (!default_context_inited)
         return;
     context_deinit(&default_context_int);
+    CRYPTO_THREAD_cleanup_local(&default_context_thread_local);
     default_context_inited = 0;
 }
 
@@ -402,8 +412,7 @@ static OSSL_LIB_CTX *get_thread_default_context(void)
     if (!RUN_ONCE(&default_context_init, default_context_do_init))
         return NULL;
 
-    return CRYPTO_THREAD_get_local_ex(CRYPTO_THREAD_LOCAL_DEF_CTX_KEY,
-                                      CRYPTO_THREAD_NO_CONTEXT);
+    return CRYPTO_THREAD_get_local(&default_context_thread_local);
 }
 
 static OSSL_LIB_CTX *get_default_context(void)
@@ -420,8 +429,7 @@ static int set_default_context(OSSL_LIB_CTX *defctx)
     if (defctx == &default_context_int)
         defctx = NULL;
 
-    return CRYPTO_THREAD_set_local_ex(CRYPTO_THREAD_LOCAL_DEF_CTX_KEY,
-                                      CRYPTO_THREAD_NO_CONTEXT, defctx);
+    return CRYPTO_THREAD_set_local(&default_context_thread_local, defctx);
 }
 #endif
 
diff --git a/include/internal/threads_common.h b/include/internal/threads_common.h
index 4b949fe956c..5992d65add2 100644
--- a/include/internal/threads_common.h
+++ b/include/internal/threads_common.h
@@ -14,7 +14,6 @@ typedef enum {
     CRYPTO_THREAD_LOCAL_RCU_KEY = 0,
     CRYPTO_THREAD_LOCAL_DRBG_PRIV_KEY,
     CRYPTO_THREAD_LOCAL_DRBG_PUB_KEY,
-    CRYPTO_THREAD_LOCAL_DEF_CTX_KEY,
     CRYPTO_THREAD_LOCAL_ERR_KEY,
     CRYPTO_THREAD_LOCAL_ASYNC_CTX_KEY,
     CRYPTO_THREAD_LOCAL_ASYNC_POOL_KEY,