commit 4966eb8ce8f8e48a5994e6ffcf93212eb4af969c
author Dave Rodgman <dave.rodgman@arm.com> Wed Dec 13 11:19:21 2023 +0000
committer GitHub <noreply@github.com> Wed Dec 13 11:19:21 2023 +0000
tree aa69f8555a8a8528a35ccd039dd5e048be932465
parent 0dc3ed029a019c789283935a6fb10e7b7769d4a2
parent ec4c47f62b73c83b0f80c8b6f7fe93448c138e83

Merge pull request #8625 from davidhorstmann-arm/fix-uninit-mpi-test-2.28

[Backport 2.28] Fix possible free of uninitialized MPI

tests/suites/test_suite_x509write.function

diff --git a/tests/suites/test_suite_x509write.function b/tests/suites/test_suite_x509write.function
index ad0f2a6..90c2192 100644
--- a/tests/suites/test_suite_x509write.function
+++ b/tests/suites/test_suite_x509write.function
@@ -435,10 +435,11 @@
     mbedtls_mpi serial_mpi;
     uint8_t invalid_serial[MBEDTLS_X509_RFC5280_MAX_SERIAL_LEN + 1];
 
+    mbedtls_mpi_init(&serial_mpi);
+
     USE_PSA_INIT();
     memset(invalid_serial, 0x01, sizeof(invalid_serial));
 
-    mbedtls_mpi_init(&serial_mpi);
     TEST_EQUAL(mbedtls_mpi_read_binary(&serial_mpi, invalid_serial,
                                        sizeof(invalid_serial)), 0);
     TEST_EQUAL(mbedtls_x509write_crt_set_serial(&ctx, &serial_mpi),