commit df823bf39b54b7057adc48e6d44123d0fcd486f0
author Ronald Cron <ronald.cron@arm.com> Tue Mar 29 18:57:54 2022 +0200
committer Ronald Cron <ronald.cron@arm.com> Tue Mar 29 18:57:54 2022 +0200
tree 337ed428da3a8e462ad7c8b90629422926ef9462
parent 42c1cbf1dece31d5c0a8d86fa38a477461635b32

ssl_client.c: Re-order partially extension writing

Signed-off-by: Ronald Cron <ronald.cron@arm.com>

library/ssl_client.c

diff --git a/library/ssl_client.c b/library/ssl_client.c
index fd96865..769a8f2 100644
--- a/library/ssl_client.c
+++ b/library/ssl_client.c
@@ -411,12 +411,13 @@
     p_extensions_len = p;
     p += 2;
 
-#if defined(MBEDTLS_SSL_PROTO_TLS1_3)
-    ret = mbedtls_ssl_tls13_write_client_hello_exts( ssl, p, end, &output_len );
+#if defined(MBEDTLS_SSL_SERVER_NAME_INDICATION)
+    /* Write server name extension */
+    ret = mbedtls_ssl_write_hostname_ext( ssl, p, end, &output_len );
     if( ret != 0 )
         return( ret );
     p += output_len;
-#endif
+#endif /* MBEDTLS_SSL_SERVER_NAME_INDICATION */
 
 #if defined(MBEDTLS_SSL_ALPN)
     ret = ssl_write_alpn_ext( ssl, p, end, &output_len );
@@ -426,6 +427,13 @@
 #endif /* MBEDTLS_SSL_ALPN */
 
 #if defined(MBEDTLS_SSL_PROTO_TLS1_3)
+    ret = mbedtls_ssl_tls13_write_client_hello_exts( ssl, p, end, &output_len );
+    if( ret != 0 )
+        return( ret );
+    p += output_len;
+#endif
+
+#if defined(MBEDTLS_SSL_PROTO_TLS1_3)
 #if defined(MBEDTLS_KEY_EXCHANGE_WITH_CERT_ENABLED)
     if( mbedtls_ssl_conf_tls13_some_ephemeral_enabled( ssl ) )
     {
@@ -445,14 +453,6 @@
 #endif /* MBEDTLS_KEY_EXCHANGE_WITH_CERT_ENABLED */
 #endif /* MBEDTLS_SSL_PROTO_TLS1_3 */
 
-#if defined(MBEDTLS_SSL_SERVER_NAME_INDICATION)
-    /* Write server name extension */
-    ret = mbedtls_ssl_write_hostname_ext( ssl, p, end, &output_len );
-    if( ret != 0 )
-        return( ret );
-    p += output_len;
-#endif /* MBEDTLS_SSL_SERVER_NAME_INDICATION */
-
     /* Add more extensions here */
 
     /* Write the length of the list of extensions. */