Adds MBEDTLS_THREADING_C implementation for Windows using srwlock

include/mbedtls/check_config.h

@@ -959,6 +959,13 @@
 #define MBEDTLS_THREADING_IMPL
 #endif
 
+#if defined(MBEDTLS_THREADING_SRWLOCK)
+#if !defined(MBEDTLS_THREADING_C) || defined(MBEDTLS_THREADING_IMPL)
+#error "MBEDTLS_THREADING_SRWLOCK defined, but not all prerequisites"
+#endif
+#define MBEDTLS_THREADING_IMPL
+#endif
+
 #if defined(MBEDTLS_THREADING_ALT)
 #if !defined(MBEDTLS_THREADING_C) || defined(MBEDTLS_THREADING_IMPL)
 #error "MBEDTLS_THREADING_ALT defined, but not all prerequisites"

include/mbedtls/mbedtls_config.h

@@ -1940,6 +1940,17 @@
  */
 //#define MBEDTLS_THREADING_PTHREAD
 
+/**
+ * \def MBEDTLS_THREADING_SRWLOCK
+ *
+ * Enable the Windows SRW Locks wrapper layer for the threading layer.
+ *
+ * Requires: MBEDTLS_THREADING_C
+ *
+ * Uncomment this to enable SRW Lock mutexes.
+ */
+//#define MBEDTLS_THREADING_SRWLOCK
+
 /**
  * \def MBEDTLS_USE_PSA_CRYPTO
  *
@@ -3307,7 +3318,7 @@
  * provided).
  *
  * You will have to enable either MBEDTLS_THREADING_ALT or
- * MBEDTLS_THREADING_PTHREAD.
+ * MBEDTLS_THREADING_PTHREAD or MBEDTLS_THREADING_SRWLOCK
  *
  * Enable this layer to allow use of mutexes within mbed TLS
  */

include/mbedtls/threading.h

@@ -47,6 +47,19 @@ typedef struct mbedtls_threading_mutex_t {
 } mbedtls_threading_mutex_t;
 #endif
 
+#if defined(MBEDTLS_THREADING_SRWLOCK)
+/*
+ * Use spaces to pass the macroses name check.
+ */
+#   define WIN32_LEAN_AND_MEAN
+#   include <windows.h>
+#   include <synchapi.h>
+#   undef WIN32_LEAN_AND_MEAN
+typedef struct mbedtls_threading_mutex_t {
+    SRWLOCK lock;
+} mbedtls_threading_mutex_t;
+#endif
+
 #if defined(MBEDTLS_THREADING_ALT)
 /* You should define the mbedtls_threading_mutex_t type in your header */
 #include "threading_alt.h"

library/threading.c

@@ -125,6 +125,60 @@ int (*mbedtls_mutex_unlock)(mbedtls_threading_mutex_t *) = threading_mutex_unloc
 
 #endif /* MBEDTLS_THREADING_PTHREAD */
 
+#if defined(MBEDTLS_THREADING_SRWLOCK)
+static void threading_mutex_init_srwlock(mbedtls_threading_mutex_t *mutex)
+{
+    if (mutex == NULL) {
+        return;
+    }
+
+    InitializeSRWLock(&mutex->lock);
+}
+
+static void threading_mutex_free_srwlock(mbedtls_threading_mutex_t *mutex)
+{
+    if (mutex == NULL) {
+        return;
+    }
+    /*
+     * SRW locks do not need to be explicitly destroyed.
+     */
+}
+
+static int threading_mutex_lock_srwlock(mbedtls_threading_mutex_t *mutex)
+{
+    if (mutex == NULL) {
+        return MBEDTLS_ERR_THREADING_BAD_INPUT_DATA;
+    }
+
+    AcquireSRWLockExclusive(&mutex->lock);
+
+    return 0;
+}
+
+static int threading_mutex_unlock_srwlock(mbedtls_threading_mutex_t *mutex)
+{
+    if (mutex == NULL) {
+        return MBEDTLS_ERR_THREADING_BAD_INPUT_DATA;
+    }
+
+    ReleaseSRWLockExclusive(&mutex->lock);
+
+    return 0;
+}
+
+void (*mbedtls_mutex_init)(mbedtls_threading_mutex_t *) = threading_mutex_init_srwlock;
+void (*mbedtls_mutex_free)(mbedtls_threading_mutex_t *) = threading_mutex_free_srwlock;
+int (*mbedtls_mutex_lock)(mbedtls_threading_mutex_t *) = threading_mutex_lock_srwlock;
+int (*mbedtls_mutex_unlock)(mbedtls_threading_mutex_t *) = threading_mutex_unlock_srwlock;
+
+/*
+ * With SRW Lock we can statically initialize mutexes
+ */
+#define MUTEX_INIT = SRWLOCK_INIT
+
+#endif /* MBEDTLS_THREADING_SRWLOCK */
+
 #if defined(MBEDTLS_THREADING_ALT)
 static int threading_mutex_fail(mbedtls_threading_mutex_t *mutex)
 {

scripts/config.py

@@ -214,6 +214,7 @@ def realfull_adapter(_name, active, section):
     'MBEDTLS_SHA512_USE_A64_CRYPTO_ONLY', # interacts with *_USE_A64_CRYPTO_IF_PRESENT
     'MBEDTLS_TEST_CONSTANT_FLOW_MEMSAN', # build dependency (clang+memsan)
     'MBEDTLS_TEST_CONSTANT_FLOW_VALGRIND', # build dependency (valgrind headers)
+    'MBEDTLS_THREADING_SRWLOCK', # os dependency (windows)
     'MBEDTLS_X509_REMOVE_INFO', # removes a feature
 ])
 

tests/scripts/all.sh

@@ -3641,7 +3641,9 @@ component_build_mingw () {
     make CC=i686-w64-mingw32-gcc AR=i686-w64-mingw32-ar LD=i686-w64-minggw32-ld CFLAGS='-Werror' WINDOWS_BUILD=1 tests
     make WINDOWS_BUILD=1 clean
 
-    msg "build: Windows cross build - mingw64, make (DLL)" # ~ 30s
+    msg "build: Windows cross build - mingw64, make (Link Library)" # ~ 30s
+    scripts/config.py set MBEDTLS_THREADING_C
+    scripts/config.py set MBEDTLS_THREADING_SRWLOCK
     make CC=i686-w64-mingw32-gcc AR=i686-w64-mingw32-ar LD=i686-w64-minggw32-ld CFLAGS='-Werror -Wall -Wextra' WINDOWS_BUILD=1 SHARED=1 lib programs
     make CC=i686-w64-mingw32-gcc AR=i686-w64-mingw32-ar LD=i686-w64-minggw32-ld CFLAGS='-Werror -Wall -Wextra' WINDOWS_BUILD=1 SHARED=1 tests
     make WINDOWS_BUILD=1 clean