commit 3a994b7dbef99db03535bcee5c8316129ea23c25
author Valerio Setti <valerio.setti@nordicsemi.no> Wed Jul 03 16:58:10 2024 +0200
committer Valerio Setti <valerio.setti@nordicsemi.no> Wed Jul 03 16:58:10 2024 +0200
tree 5b5b132b6a607f0b560312fc4b964486e1639fc0
parent fb2da6fb354e5bbd2bbf5300a3533ba5ad99e0cd

tests_suite_debug: fix psa initialization

Since MD_OR_USE_PSA_INIT() can fail and jump to the "exit"
label it should be placed after all initializations has been
done. This issue was discovered by Coverity testing.

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>

tests/suites/test_suite_debug.function

diff --git a/tests/suites/test_suite_debug.function b/tests/suites/test_suite_debug.function
index 982be3c..5cd02b9 100644
--- a/tests/suites/test_suite_debug.function
+++ b/tests/suites/test_suite_debug.function
@@ -59,10 +59,9 @@
     mbedtls_ssl_config conf;
     struct buffer_data buffer;
 
-    MD_OR_USE_PSA_INIT();
-
     mbedtls_ssl_init(&ssl);
     mbedtls_ssl_config_init(&conf);
+    MD_OR_USE_PSA_INIT();
     memset(buffer.buf, 0, 2000);
     buffer.ptr = buffer.buf;
 
@@ -98,10 +97,9 @@
     mbedtls_ssl_config conf;
     struct buffer_data buffer;
 
-    MD_OR_USE_PSA_INIT();
-
     mbedtls_ssl_init(&ssl);
     mbedtls_ssl_config_init(&conf);
+    MD_OR_USE_PSA_INIT();
     memset(buffer.buf, 0, 2000);
     buffer.ptr = buffer.buf;
 
@@ -134,10 +132,9 @@
     mbedtls_ssl_config conf;
     struct buffer_data buffer;
 
-    MD_OR_USE_PSA_INIT();
-
     mbedtls_ssl_init(&ssl);
     mbedtls_ssl_config_init(&conf);
+    MD_OR_USE_PSA_INIT();
     memset(buffer.buf, 0, 2000);
     buffer.ptr = buffer.buf;
 
@@ -211,11 +208,10 @@
     struct buffer_data buffer;
     mbedtls_mpi val;
 
-    MD_OR_USE_PSA_INIT();
-
     mbedtls_ssl_init(&ssl);
     mbedtls_ssl_config_init(&conf);
     mbedtls_mpi_init(&val);
+    MD_OR_USE_PSA_INIT();
     memset(buffer.buf, 0, 2000);
     buffer.ptr = buffer.buf;