Initialize RSA context in RSA test suite before first potentially failing operation

The function `rsa_gen_key` from `test_suite_rsa.function` initialized a stack allocated RSA context only after
seeding the CTR DRBG. If the latter operation failed, the cleanup code tried to free the uninitialized RSA context,
potentially resulting in a segmentation fault. Fixes one aspect of #1023.
This commit is contained in:
Hanno Becker 2017-09-08 10:35:32 +01:00
parent a75a459143
commit 8cd5d43639

View file

@ -667,12 +667,12 @@ void rsa_gen_key( int nrbits, int exponent, int result)
const char *pers = "test_suite_rsa";
entropy_init( &entropy );
rsa_init( &ctx, 0, 0 );
TEST_ASSERT( ctr_drbg_init( &ctr_drbg, entropy_func, &entropy,
(const unsigned char *) pers,
strlen( pers ) ) == 0 );
rsa_init( &ctx, 0, 0 );
TEST_ASSERT( rsa_gen_key( &ctx, ctr_drbg_random, &ctr_drbg, nrbits,
exponent ) == result );
if( result == 0 )