commit 63e89b46f8b6cd603bb2b73e58545f940aae9a85
author Dave Rodgman <dave.rodgman@arm.com> Wed Jun 21 11:55:17 2023 +0100
committer Dave Rodgman <dave.rodgman@arm.com> Wed Jun 21 11:58:54 2023 +0100
tree 1710431a304fc00e403735ac6b71fbd367b6a085
parent 85842b8edb1f3cf18cb525f068d9c105b09563fe

Use UINTPTR_MAX not SIZE_MAX

Signed-off-by: Dave Rodgman <dave.rodgman@arm.com>

library/constant_time.c

diff --git a/library/constant_time.c b/library/constant_time.c
index fb14c9c..f7da39f 100644
--- a/library/constant_time.c
+++ b/library/constant_time.c
@@ -63,8 +63,9 @@
  * only used here.
  */
 #if defined(MBEDTLS_EFFICIENT_UNALIGNED_ACCESS) && defined(MBEDTLS_HAVE_ASM)
-#if ((defined(__arm__) || defined(__thumb__) || defined(__thumb2__)) && (SIZE_MAX == 0xffffffff)) || \
-    (defined(__aarch64__) && ((SIZE_MAX == 0xffffffff) || (SIZE_MAX == 0xffffffffffffffff)))
+#if ((defined(__arm__) || defined(__thumb__) || defined(__thumb2__)) && (UINTPTR_MAX == 0xfffffffful)) || \
+    (defined(__aarch64__) && ((UINTPTR_MAX == 0xffffffffull) || (UINTPTR_MAX == 0xffffffffffffffffull)))
+/* We check pointer sizes to avoid issues with them not matching register size requirements */
 #define MBEDTLS_EFFICIENT_UNALIGNED_VOLATILE_ACCESS
 #endif
 #endif
@@ -80,10 +81,11 @@
 #if defined(__arm__) || defined(__thumb__) || defined(__thumb2__)
     asm volatile ("ldr %0, [%1]" : "=r" (r) : "r" (p) :);
 #elif defined(__aarch64__)
-#if (SIZE_MAX == 0xffffffff)
+#if (UINTPTR_MAX == 0xfffffffful)
     /* ILP32: Specify the pointer operand slightly differently, as per #7787. */
     asm volatile ("ldr %w0, [%1]" : "=r" (r) : "p" (p) :);
-#else
+#elif (UINTPTR_MAX == 0xffffffffffffffffull)
+    /* aarch64 with 64-bit pointers */
     asm volatile ("ldr %w0, [%1]" : "=r" (r) : "r" (p) :);
 #endif
 #endif