From 505712338ea7ac8661026c2acd8f8ccb310c7e01 Mon Sep 17 00:00:00 2001 From: Przemyslaw Stekiel Date: Wed, 3 Nov 2021 14:19:52 +0100 Subject: [PATCH] ssl_client2: move memory leak check before rng_free() Signed-off-by: Przemyslaw Stekiel --- programs/ssl/ssl_client2.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/programs/ssl/ssl_client2.c b/programs/ssl/ssl_client2.c index 62bba1380..4360fd343 100644 --- a/programs/ssl/ssl_client2.c +++ b/programs/ssl/ssl_client2.c @@ -3049,6 +3049,10 @@ exit: #endif /* MBEDTLS_KEY_EXCHANGE_SOME_PSK_ENABLED && MBEDTLS_USE_PSA_CRYPTO */ + mbedtls_ssl_session_free( &saved_session ); + mbedtls_ssl_free( &ssl ); + mbedtls_ssl_config_free( &conf ); + #if defined(MBEDTLS_USE_PSA_CRYPTO) const char* message = mbedtls_test_helper_is_psa_leaking(); if( message ) @@ -3066,9 +3070,6 @@ exit: mbedtls_psa_crypto_free( ); #endif - mbedtls_ssl_session_free( &saved_session ); - mbedtls_ssl_free( &ssl ); - mbedtls_ssl_config_free( &conf ); rng_free( &rng ); if( session_data != NULL ) mbedtls_platform_zeroize( session_data, session_data_len );