]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUILD: ssl: address a recent build warning when QUIC is enabled
authorWilly Tarreau <w@1wt.eu>
Wed, 10 Sep 2025 08:43:08 +0000 (10:43 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 10 Sep 2025 08:44:33 +0000 (10:44 +0200)
Since commit 5ab9954faa ("MINOR: ssl: Add a flag to let it known we have
an ALPN negociated"), when building with QUIC we get this warning:

  src/ssl_sock.c: In function 'ssl_sock_advertise_alpn_protos':
  src/ssl_sock.c:2189:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]

Let's just move the instructions after the optional declaration. No
backport is needed.

src/ssl_sock.c

index f8640e2f140fefd66d2e41c0dcb7b109c6b16adb..69dd4e70eab2614680a0092051458b0e102db11a 100644 (file)
@@ -2181,13 +2181,11 @@ static int ssl_sock_advertise_alpn_protos(SSL *s, const unsigned char **out,
        struct connection *conn;
        struct ssl_sock_ctx *ctx;
 
-       conn = SSL_get_ex_data(s, ssl_app_data_index);
-       ctx = __conn_get_ssl_sock_ctx(conn);
-
-
 #ifdef USE_QUIC
        struct quic_conn *qc = SSL_get_ex_data(s, ssl_qc_app_data_index);
 #endif
+       conn = SSL_get_ex_data(s, ssl_app_data_index);
+       ctx = __conn_get_ssl_sock_ctx(conn);
 
        if (SSL_select_next_proto((unsigned char**) out, outlen, (const unsigned char *)conf->alpn_str,
                                  conf->alpn_len, server, server_len) != OPENSSL_NPN_NEGOTIATED) {