| commit | c69fa50d4c6d6fd955d38ed588c7a5ab997121aa | [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 Mar 09 21:06:19 2016 +0000 |
| tree | 16694a05dd006e5730acf51b65e3a7deb7e3e23c | |
| parent | 8a49a019b0daf508d21d9ef3ca1a60a49ae9a126 [diff] [blame] |
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 2baf532..4c85c29 100644 --- a/library/rsa.c +++ b/library/rsa.c
@@ -856,8 +856,7 @@ bad |= *p++; /* Must be zero */ } - if( pad_count < 8 ) - return( MBEDTLS_ERR_RSA_BAD_INPUT_DATA ); + bad |= ( pad_count < 8 ); if( bad ) return( MBEDTLS_ERR_RSA_INVALID_PADDING );