commit 4ca9140d433afaf254e533a5cf92db6461c97892
author Jerry Yu <jerry.h.yu@arm.com> Mon May 09 15:50:57 2022 +0800
committer Jerry Yu <jerry.h.yu@arm.com> Mon May 09 15:55:39 2022 +0800
tree 088cd739d6be2da8813f9d136edef100ae4fa32f
parent 66d9e6f4058a925cd1c76c1ffa5f9b6d2e342d59

fix coding style issues


Signed-off-by: Jerry Yu <jerry.h.yu@arm.com>

library/ssl_tls13_server.c

diff --git a/library/ssl_tls13_server.c b/library/ssl_tls13_server.c
index baf8c61..d68403f 100644
--- a/library/ssl_tls13_server.c
+++ b/library/ssl_tls13_server.c
@@ -730,7 +730,8 @@
     int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
     unsigned char* buf = NULL;
     size_t buflen = 0;
-    int parse_client_hello_ret ;
+    int parse_client_hello_ret;
+
     MBEDTLS_SSL_DEBUG_MSG( 2, ( "=> parse client hello" ) );
 
     MBEDTLS_SSL_PROC_CHK( mbedtls_ssl_tls13_fetch_handshake_msg(
@@ -739,25 +740,14 @@
 
     MBEDTLS_SSL_PROC_CHK_NEG( ssl_tls13_parse_client_hello( ssl, buf,
                                                             buf + buflen ) );
-
     parse_client_hello_ret = ret;
+
     MBEDTLS_SSL_PROC_CHK( ssl_tls13_postprocess_client_hello( ssl ) );
 
-    switch( parse_client_hello_ret )
-    {
-        case SSL_CLIENT_HELLO_OK:
-            mbedtls_ssl_handshake_set_state( ssl, MBEDTLS_SSL_SERVER_HELLO );
-            break;
-
-        case SSL_CLIENT_HELLO_HRR_REQUIRED:
-            mbedtls_ssl_handshake_set_state( ssl, MBEDTLS_SSL_HELLO_RETRY_REQUEST );
-            break;
-
-        default:
-            MBEDTLS_SSL_DEBUG_MSG( 2, ( "should never happen" ) );
-            ret = parse_client_hello_ret;
-            break;
-    }
+    if( parse_client_hello_ret == SSL_CLIENT_HELLO_OK )
+        mbedtls_ssl_handshake_set_state( ssl, MBEDTLS_SSL_SERVER_HELLO );
+    else
+        mbedtls_ssl_handshake_set_state( ssl, MBEDTLS_SSL_HELLO_RETRY_REQUEST );
 
 cleanup: