commit 918708b824bd44ba333e0e5296ee77034252d1eb
author Gilles Peskine <gilles.peskine@arm.com> Fri Jun 25 13:43:56 2021 +0200
committer GitHub <noreply@github.com> Fri Jun 25 13:43:56 2021 +0200
tree b83ec4de994eff2f5453c301d90e881eb145f5dd
parent 3698fa1043c290e13279d422b98f6e62d4695cbe
parent e57dd2d4500c9d994dde97fc10c5eeb6e1b211db

Merge pull request #4729 from paul-elliott-arm/fix_test_illegal_access

Fix potential free of uninitialised pointer

tests/suites/test_suite_rsa.function

diff --git a/tests/suites/test_suite_rsa.function b/tests/suites/test_suite_rsa.function
index f5f4aac..f5f5159 100644
--- a/tests/suites/test_suite_rsa.function
+++ b/tests/suites/test_suite_rsa.function
@@ -179,11 +179,12 @@
     mbedtls_test_rnd_pseudo_info rnd_info;
 
     mbedtls_rsa_init( &ctx );
-    TEST_ASSERT( mbedtls_rsa_set_padding( &ctx, padding_mode,
-                                          MBEDTLS_MD_NONE ) == 0 );
     mbedtls_mpi_init( &N ); mbedtls_mpi_init( &P );
     mbedtls_mpi_init( &Q ); mbedtls_mpi_init( &E );
 
+    TEST_ASSERT( mbedtls_rsa_set_padding( &ctx, padding_mode,
+                                          MBEDTLS_MD_NONE ) == 0 );
+
     memset( output, 0x00, sizeof( output ) );
     memset( &rnd_info, 0, sizeof( mbedtls_test_rnd_pseudo_info ) );