commit a7eb81290b178ad74adf27b5ac33060087e050b1
author Janos Follath <janos.follath@arm.com> Tue Aug 20 12:33:42 2024 +0100
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Thu Sep 05 10:46:50 2024 +0200
tree 4eb002a40702343a89a10a71b3de6d9bbb803937
parent 76c0e6f3a255c58c1e52d74fcdc7e2b8a157bdd7

Fix memory corruption in exp_mod tests

Signed-off-by: Janos Follath <janos.follath@arm.com>

tf-psa-crypto/tests/suites/test_suite_bignum_core.function

diff --git a/tf-psa-crypto/tests/suites/test_suite_bignum_core.function b/tf-psa-crypto/tests/suites/test_suite_bignum_core.function
index 39a5052..3128307 100644
--- a/tf-psa-crypto/tests/suites/test_suite_bignum_core.function
+++ b/tf-psa-crypto/tests/suites/test_suite_bignum_core.function
@@ -1329,7 +1329,7 @@
     /* Check both with output aliased to input */
 
     TEST_CALLOC(A_copy, A_limbs);
-    memcpy(A_copy, A, sizeof(A_copy) * A_limbs);
+    memcpy(A_copy, A, sizeof(*A_copy) * A_limbs);
 
     TEST_CF_SECRET(A, A_limbs * sizeof(mbedtls_mpi_uint));
     TEST_CF_SECRET(N, N_limbs * sizeof(mbedtls_mpi_uint));
@@ -1344,7 +1344,7 @@
     TEST_EQUAL(0, memcmp(X, A, N_limbs * sizeof(mbedtls_mpi_uint)));
 
     TEST_CF_PUBLIC(A, A_limbs * sizeof(mbedtls_mpi_uint));
-    memcpy(A, A_copy, sizeof(A) * A_limbs);
+    memcpy(A, A_copy, sizeof(*A) * A_limbs);
 #if defined(MBEDTLS_TEST_HOOKS) && !defined(MBEDTLS_THREADING_C)
     mbedtls_mpi_optionally_safe_codepath_reset();
 #endif