commit 9e2237ac473e31e12d44a9550ac27583dd79d884
author Hanno Becker <hanno.becker@arm.com> Thu Aug 16 15:51:34 2018 +0100
committer Hanno Becker <hanno.becker@arm.com> Thu Aug 16 15:53:17 2018 +0100
tree 90dc7203ab13c21ab4b5c234dbbb0347cc3cbace
parent 942cfea65f86e839a127dba2a3e286738ff4e47c

Fix overly strict bounds check in ssl_parse_certificate_request()

library/ssl_cli.c

diff --git a/library/ssl_cli.c b/library/ssl_cli.c
index 916a29c..4d8288e 100644
--- a/library/ssl_cli.c
+++ b/library/ssl_cli.c
@@ -2504,7 +2504,7 @@
      * therefore the buffer length at this point must be greater than that
      * regardless of the actual code path.
      */
-    if( ssl->in_hslen <= mbedtls_ssl_hs_hdr_len( ssl ) + 3 + n )
+    if( ssl->in_hslen <= mbedtls_ssl_hs_hdr_len( ssl ) + 2 + n )
     {
         MBEDTLS_SSL_DEBUG_MSG( 1, ( "bad certificate request message" ) );
         return( MBEDTLS_ERR_SSL_BAD_HS_CERTIFICATE_REQUEST );