| commit | f18263d78bcab30df0401463d04159a2c0ac098b | [log] [tgz] |
|---|---|---|
| author | Janos Follath <janos.follath@arm.com> | Fri Feb 12 13:30:09 2016 +0000 |
| committer | Simon Butcher <simon.butcher@arm.com> | Wed May 18 19:58:40 2016 +0100 |
| tree | 1c36951dffa8a381006cb56bbe3d5059416fdc0d | |
| parent | 7244ecf52e769debd1f91eec43b7f0d445efd65e [diff] |
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.
diff --git a/library/rsa.c b/library/rsa.c index e9ada0e..fd33b53 100644 --- a/library/rsa.c +++ b/library/rsa.c
@@ -851,8 +851,7 @@ bad |= *p++; /* Must be zero */ } - if( pad_count < 8 ) - return( MBEDTLS_ERR_RSA_BAD_INPUT_DATA ); + bad |= ( pad_count < 8 ); if( bad ) return( POLARSSL_ERR_RSA_INVALID_PADDING );