From 29fe76b84356144d6452393a069d8b5636d385c0 Mon Sep 17 00:00:00 2001 From: Paul Elliott Date: Thu, 24 Oct 2024 14:27:26 +0100 Subject: [PATCH] Fix tests where tests were done prior to init Variables that are in any way destructed on exit should be initialised prior to any tests that might jump to exit, to save potential uninitialised memory accesses. Signed-off-by: Paul Elliott --- tests/suites/test_suite_ctr_drbg.function | 6 +++--- tests/suites/test_suite_x509write.function | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/suites/test_suite_ctr_drbg.function b/tests/suites/test_suite_ctr_drbg.function index 9fa55a754b..a19dea3609 100644 --- a/tests/suites/test_suite_ctr_drbg.function +++ b/tests/suites/test_suite_ctr_drbg.function @@ -365,12 +365,12 @@ void ctr_drbg_threads(data_t *expected_result, int reseed, int arg_thread_count) AES_PSA_INIT(); - TEST_CALLOC(threads, sizeof(mbedtls_test_thread_t) * thread_count); - memset(out, 0, sizeof(out)); - mbedtls_ctr_drbg_context ctx; mbedtls_ctr_drbg_init(&ctx); + TEST_CALLOC(threads, sizeof(mbedtls_test_thread_t) * thread_count); + memset(out, 0, sizeof(out)); + test_offset_idx = 0; /* Need to set a non-default fixed entropy len, to ensure same output across diff --git a/tests/suites/test_suite_x509write.function b/tests/suites/test_suite_x509write.function index 2762b0f84e..8726dec3ab 100644 --- a/tests/suites/test_suite_x509write.function +++ b/tests/suites/test_suite_x509write.function @@ -297,11 +297,12 @@ void x509_csr_check_opaque(char *key_file, int md_type, int key_usage, mbedtls_test_rnd_pseudo_info rnd_info; mbedtls_x509write_csr_init(&req); + mbedtls_pk_init(&key); + MD_OR_USE_PSA_INIT(); memset(&rnd_info, 0x2a, sizeof(mbedtls_test_rnd_pseudo_info)); - mbedtls_pk_init(&key); TEST_ASSERT(mbedtls_pk_parse_keyfile(&key, key_file, NULL, mbedtls_test_rnd_std_rand, NULL) == 0);