commit a05cbecc909e1ab95a68056d4c45eacec340c780
author Cédric Meuter <cedric.meuter@gmail.com> Sat Apr 25 15:02:34 2020 +0200
committer Cédric Meuter <cedric.meuter@gmail.com> Thu Dec 24 10:07:41 2020 +0100
tree bc4a91e4910547cf238faa294dcdf0c7cdcd70da
parent f3fab3314747a2f1948ce9ecc224ff55e9bbb2fe

Added tests for mbedtls_rsa_rsassa_pss_sign_ext()

- added some invalid param tests in test_suite_rsa
- added functional tests in test_suite_pkcs1_v21

Signed-off-by: Cédric Meuter <cedric.meuter@gmail.com>

library/rsa.c

diff --git a/library/rsa.c b/library/rsa.c
index 2b4b0fd..02423c0 100644
--- a/library/rsa.c
+++ b/library/rsa.c
@@ -1812,6 +1812,8 @@
                         hashlen == 0 ) ||
                       hash != NULL );
     RSA_VALIDATE_RET( sig != NULL );
+    RSA_VALIDATE_RET( saltlen == MBEDTLS_RSA_SALT_LEN_ANY ||
+                      saltlen > 0 );
 
     if( mode == MBEDTLS_RSA_PRIVATE && ctx->padding != MBEDTLS_RSA_PKCS_V21 )
         return( MBEDTLS_ERR_RSA_BAD_INPUT_DATA );
@@ -1854,7 +1856,7 @@
         else
             slen = olen - hlen - 2;
     }
-    else if ( (saltlen < 0) || ((size_t) saltlen > olen - hlen - 2) )
+    else if ( ( (size_t) saltlen ) > olen - hlen - 2 )
     {
         return( MBEDTLS_ERR_RSA_BAD_INPUT_DATA );
     }