commit 4e8cd7b903aecb431bd0792282ef056c6f5fa1fa
author XiaokangQian <xiaokang.qian@arm.com> Thu Apr 21 09:48:09 2022 +0000
committer XiaokangQian <xiaokang.qian@arm.com> Thu Apr 21 13:30:18 2022 +0000
tree 44e5c7305b7665bfe44564a369af9df1533c680c
parent 060d8675983645c1c207736c6b282645d29012a3

Remove useless selected_group

Change-Id: I5fb76b5bf4b22d0231c17314783781f9e7c309a3
Signed-off-by: XiaokangQian <xiaokang.qian@arm.com>

library/ssl_tls13_server.c

diff --git a/library/ssl_tls13_server.c b/library/ssl_tls13_server.c
index 1a96eaf..5f9b5d0 100644
--- a/library/ssl_tls13_server.c
+++ b/library/ssl_tls13_server.c
@@ -115,7 +115,7 @@
     p += 2;
     MBEDTLS_SSL_CHK_BUF_READ_PTR( p, end, named_group_list_len );
     named_group_list_end = p + named_group_list_len;
-    ssl->handshake->selected_group = 0;
+    ssl->handshake->hrr_selected_group = 0;
 
     while( p < named_group_list_end )
     {
@@ -128,7 +128,7 @@
 
         if( ! mbedtls_ssl_named_group_is_offered( ssl, named_group ) ||
             ! mbedtls_ssl_named_group_is_supported( named_group ) ||
-            ssl->handshake->selected_group != 0 )
+            ssl->handshake->hrr_selected_group != 0 )
         {
             continue;
         }
@@ -136,7 +136,7 @@
         MBEDTLS_SSL_DEBUG_MSG(
                 2, ( "add named group (%04x) into received list.",
                      named_group ) );
-        ssl->handshake->selected_group = named_group;
+        ssl->handshake->hrr_selected_group = named_group;
     }
 
     return( 0 );
@@ -220,20 +220,22 @@
         {
             continue;
         }
-        const mbedtls_ecp_curve_info *curve_info =
-                mbedtls_ecp_curve_info_from_tls_id( group );
 
         /*
          * For now, we only support ECDHE groups.
          */
         if( mbedtls_ssl_tls13_named_group_is_ecdhe( group ) )
         {
-            match_found = 1;
+            const mbedtls_ecp_curve_info *curve_info =
+                    mbedtls_ecp_curve_info_from_tls_id( group );
+            ((void) curve_info);
             MBEDTLS_SSL_DEBUG_MSG( 2, ( "ECDH curve: %s", curve_info->name ) );
             ret = mbedtls_ssl_tls13_read_public_ecdhe_share(
                     ssl, p - 2, key_exchange_len + 2 );
             if( ret != 0 )
                 return( ret );
+
+            match_found = 1;
         }
         else
         {