commit e99b24dd9ffc9bf67a8e1c804120d376a82d2822
author Dave Rodgman <dave.rodgman@arm.com> Thu Sep 14 15:45:03 2023 +0100
committer Dave Rodgman <dave.rodgman@arm.com> Thu Sep 14 15:52:02 2023 +0100
tree 9173e1be174a1f6ad501054ede0e9de6bea19d28
parent b95e92cd41df883f2c3988ca141e7cb6a4157ffa

Fix some clang-18 warnings

Signed-off-by: Dave Rodgman <dave.rodgman@arm.com>

library/ssl_client.c

diff --git a/library/ssl_client.c b/library/ssl_client.c
index 7114ef0..d553997 100644
--- a/library/ssl_client.c
+++ b/library/ssl_client.c
@@ -648,14 +648,15 @@
           MBEDTLS_SSL_TLS1_3_KEY_EXCHANGE_MODE_SOME_EPHEMERAL_ENABLED */
 
 #if defined(MBEDTLS_SSL_HANDSHAKE_WITH_CERT_ENABLED)
-    if (
+    int write_sig_alg_ext = 0;
 #if defined(MBEDTLS_SSL_PROTO_TLS1_3)
-        (propose_tls13 && mbedtls_ssl_conf_tls13_ephemeral_enabled(ssl)) ||
+    write_sig_alg_ext = write_sig_alg_ext || (propose_tls13 && mbedtls_ssl_conf_tls13_ephemeral_enabled(ssl));
 #endif
 #if defined(MBEDTLS_SSL_PROTO_TLS1_2)
-        propose_tls12 ||
+    write_sig_alg_ext = write_sig_alg_ext || propose_tls12;
 #endif
-        0) {
+
+    if (write_sig_alg_ext) {
         ret = mbedtls_ssl_write_sig_alg_ext(ssl, p, end, &output_len);
         if (ret != 0) {
             return ret;