commit d012084e91bb12a2743eee0fa10e7a2bc284feed
author Yanray Wang <yanray.wang@arm.com> Thu Nov 23 16:35:54 2023 +0800
committer Yanray Wang <yanray.wang@arm.com> Thu Nov 23 16:38:20 2023 +0800
tree 9a4dd2a3256cf5fc52ecb07be4041fc2e144186e
parent 554ee62fbae40e98a34721a1b9591c9e9b42062b

tls13: early_data: cli: optimize code

- remove unnecessary check
- using local variable session

Signed-off-by: Yanray Wang <yanray.wang@arm.com>

library/ssl_tls13_client.c

diff --git a/library/ssl_tls13_client.c b/library/ssl_tls13_client.c
index 068676c..a62d2bc 100644
--- a/library/ssl_tls13_client.c
+++ b/library/ssl_tls13_client.c
@@ -2662,19 +2662,18 @@
                                               const unsigned char *buf,
                                               const unsigned char *end)
 {
+    mbedtls_ssl_session *session = ssl->session;
+
     MBEDTLS_SSL_CHK_BUF_READ_PTR(buf, end, 4);
 
-    if (ssl->session != NULL) {
-        ssl->session->max_early_data_size = MBEDTLS_GET_UINT32_BE(buf, 0);
-        mbedtls_ssl_session_set_ticket_flags(
-            ssl->session, MBEDTLS_SSL_TLS1_3_TICKET_ALLOW_EARLY_DATA);
-        MBEDTLS_SSL_DEBUG_MSG(
-            3, ("received max_early_data_size: %u",
-                (unsigned int) ssl->session->max_early_data_size));
-        return 0;
-    }
+    session->max_early_data_size = MBEDTLS_GET_UINT32_BE(buf, 0);
+    mbedtls_ssl_session_set_ticket_flags(
+        session, MBEDTLS_SSL_TLS1_3_TICKET_ALLOW_EARLY_DATA);
+    MBEDTLS_SSL_DEBUG_MSG(
+        3, ("received max_early_data_size: %u",
+            (unsigned int) session->max_early_data_size));
 
-    return MBEDTLS_ERR_SSL_BAD_INPUT_DATA;
+    return 0;
 }
 #endif /* MBEDTLS_SSL_EARLY_DATA */