Correct memory leak in RSA self test

The RSA self test didn't free the RSA context on failure.
This commit is contained in:
Hanno Becker 2017-05-03 15:09:31 +01:00
parent d43764f9d3
commit b81fcd00e6

View File

@ -1790,7 +1790,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( verbose != 0 ) if( verbose != 0 )
@ -1804,7 +1805,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( verbose != 0 ) if( verbose != 0 )
@ -1817,7 +1819,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( memcmp( rsa_decrypted, rsa_plaintext, len ) != 0 ) if( memcmp( rsa_decrypted, rsa_plaintext, len ) != 0 )
@ -1825,7 +1828,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( verbose != 0 ) if( verbose != 0 )
@ -1843,7 +1847,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( verbose != 0 ) if( verbose != 0 )
@ -1855,7 +1860,8 @@ int mbedtls_rsa_self_test( int verbose )
if( verbose != 0 ) if( verbose != 0 )
mbedtls_printf( "failed\n" ); mbedtls_printf( "failed\n" );
return( 1 ); ret = 1;
goto cleanup;
} }
if( verbose != 0 ) if( verbose != 0 )