From: Christopher Faulet <cfaulet@haproxy.com>
Date: Mon, 27 Apr 2020 09:17:10 +0000 (+0200)
Subject: BUG/MINOR: server: Fix server_finalize_init() to avoid unused variable
X-Git-Tag: v2.2-dev7~75
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b3b53524addbea79f5928b0bd5c58fd201a3e828;p=thirdparty%2Fhaproxy.git

BUG/MINOR: server: Fix server_finalize_init() to avoid unused variable

The variable 'ret' must only be declared When HAProxy is compiled with the SSL
support (more precisely SSL_CTRL_SET_TLSEXT_HOSTNAME must be defined).

No backport needed.
---

diff --git a/src/server.c b/src/server.c
index 14eb41f9d9..4d5e706d3a 100644
--- a/src/server.c
+++ b/src/server.c
@@ -1805,7 +1805,9 @@ static int server_sni_expr_init(const char *file, int linenum, char **args, int
 static int server_finalize_init(const char *file, int linenum, char **args, int cur_arg,
                                 struct server *srv, struct proxy *px)
 {
+#ifdef SSL_CTRL_SET_TLSEXT_HOSTNAME
 	int ret;
+#endif
 
 	if (srv->do_check && srv->trackit) {
 		ha_alert("parsing [%s:%d]: unable to enable checks and tracking at the same time!\n",