Removing 'if' branch from the fix.

This new error shouldn't be distinguishable from other padding errors.
Updating 'bad' instead of adding a new 'if' branch.
This commit is contained in:
Janos Follath 2016-02-12 13:30:09 +00:00 committed by Simon Butcher
parent f1225eaffc
commit 7c3be6b97e

View File

@ -844,8 +844,7 @@ int rsa_rsaes_pkcs1_v15_decrypt( rsa_context *ctx,
bad |= *p++; /* Must be zero */ bad |= *p++; /* Must be zero */
} }
if( pad_count < 8 ) bad |= ( pad_count < 8 );
return( MBEDTLS_ERR_RSA_BAD_INPUT_DATA );
if( bad ) if( bad )
return( POLARSSL_ERR_RSA_INVALID_PADDING ); return( POLARSSL_ERR_RSA_INVALID_PADDING );