commit 4353d3d593cda3849a04c676288e0c790fa86a3c
author Andrzej Kurek <andrzej.kurek@arm.com> Wed Jun 08 11:53:59 2022 -0400
committer Andrzej Kurek <andrzej.kurek@arm.com> Fri Jun 17 07:37:14 2022 -0400
tree 8181a457d711dfdb9ddfa8cca1e54489da4e3b63
parent 33f41a8fa858128d5e9a1d1e1afbd4e63f24d444

Split a debug message into two - for clarity
Signed-off-by: Andrzej Kurek <andrzej.kurek@arm.com>

library/ssl_srv.c

diff --git a/library/ssl_srv.c b/library/ssl_srv.c
index d6be1e1..1733ec9 100644
--- a/library/ssl_srv.c
+++ b/library/ssl_srv.c
@@ -1604,9 +1604,14 @@
     MBEDTLS_SSL_DEBUG_MSG( 3, ( "client hello v3, handshake len.: %d",
                    ( buf[1] << 16 ) | ( buf[2] << 8 ) | buf[3] ) );
 
+    if( buf[1] != 0 )
+    {
+        MBEDTLS_SSL_DEBUG_MSG( 1, ( "bad client hello message: %u != 0",
+                                    (unsigned) buf[1] ) );
+        return( MBEDTLS_ERR_SSL_BAD_HS_CLIENT_HELLO );
+    }
     /* We don't support fragmentation of ClientHello (yet?) */
-    if( buf[1] != 0 ||
-        msg_len != mbedtls_ssl_hs_hdr_len( ssl ) + ( ( buf[2] << 8 ) | buf[3] ) )
+    if( msg_len != mbedtls_ssl_hs_hdr_len( ssl ) + ( ( buf[2] << 8 ) | buf[3] ) )
     {
         MBEDTLS_SSL_DEBUG_MSG( 1, ( "bad client hello message: %u != %u + %u",
                                     (unsigned) msg_len,