RepoMirrors
/
haproxy
mirror of http://git.haproxy.org/git/haproxy.git/
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

BUG/MINOR: server: Don't rely on last default-server to init server SSL context 

During post-parsing stage, the SSL context of a server is initialized if SSL
is configured on the server or its default-server. It is required to be able
to enable SSL at runtime. However a regression was introduced, because the
last parsed default-server is used. But it is not necessarily the
default-server line used to configure the server. This may lead to
erroneously initialize the SSL context for a server without SSL parameter or
the skip it while it should be done.

The problem is the default-server used to configure a server is not saved
during configuration parsing. So, the information is lost during the
post-parsing. To fix the bug, the SRV_F_DEFSRV_USE_SSL flag is
introduced. It is used to know when a server was initialized with a
default-server using SSL.

For the record, the commit f63704488e ("MEDIUM: cli/ssl: configure ssl on
server at runtime") has introduced the bug.

This patch must be backported as far as 2.4.
This commit is contained in:
Christopher Faulet 2021-12-01 09:50:41 +01:00
parent a8ecbc4544
commit 4ab2679689
4 changed files with 22 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
include/haproxy/server-t.h
View File

@ -148,6 +148,7 @@ enum srv_initaddr {
#define SRV_F_NO_RESOLUTION 0x0800 /* disable runtime DNS resolution on this server */
#define SRV_F_DYNAMIC 0x1000 /* dynamic server instantiated at runtime */
#define SRV_F_NON_PURGEABLE 0x2000 /* this server cannot be removed at runtime */
#define SRV_F_DEFSRV_USE_SSL 0x4000 /* default-server uses SSL */
/* configured server options for send-proxy (server->pp_opts) */
#define SRV_PP_V1 0x0001 /* proxy protocol version 1 */

24
reg-tests/server/cli_set_ssl.vtc
View File

@ -26,8 +26,9 @@ haproxy h1 -conf {
default_backend test0
backend test0
default-server ssl
server www0 ${s1_addr}:${s1_port} no-ssl
default-server ssl
server www1 ${s1_addr}:${s1_port} no-ssl
backend test1
server www0 ${s1_addr}:${s1_port} no-ssl
@ -36,17 +37,22 @@ haproxy h1 -conf {
haproxy h1 -cli {
# supported case
send "show servers state test0"
expect ~ "test0 2 www1 ${s1_addr} .* - ${s1_port} - -1"
send "set server test0/www1 ssl on"
expect ~ "server ssl setting updated"
send "show servers state test0"
expect ~ "test0 2 www1 ${s1_addr} .* - ${s1_port} - 1"
send "set server test0/www1 ssl off"
expect ~ "server ssl setting updated"
send "show servers state test0"
expect ~ "test0 2 www1 ${s1_addr} .* - ${s1_port} - 0"
# unsupported cases
send "show servers state test0"
expect ~ "test0 1 www0 ${s1_addr} .* - ${s1_port} - -1"
send "set server test0/www0 ssl on"
expect ~ "server ssl setting updated"
send "show servers state test0"
expect ~ "test0 1 www0 ${s1_addr} .* - ${s1_port} - 1"
send "set server test0/www0 ssl off"
expect ~ "server ssl setting updated"
send "show servers state test0"
expect ~ "test0 1 www0 ${s1_addr} .* - ${s1_port} - 0"
expect ~ "'set server <srv> ssl' cannot be set"
# unsupported case
send "show servers state test1"
expect ~ "test1 1 www0 ${s1_addr} .* - ${s1_port} - -1"
send "set server test1/www0 ssl on"

2
src/cfgparse.c
View File

@ -3400,7 +3400,7 @@ int check_config_validity()
* without activating it */
if (newsrv->use_ssl == 1 || newsrv->check.use_ssl == 1 ||
(newsrv->proxy->options & PR_O_TCPCHK_SSL) ||
(newsrv->use_ssl != 1 && curproxy->defsrv.use_ssl == 1)) {
((newsrv->flags & SRV_F_DEFSRV_USE_SSL) && newsrv->use_ssl != 1)) {
if (xprt_get(XPRT_SSL) && xprt_get(XPRT_SSL)->prepare_srv)
cfgerr += xprt_get(XPRT_SSL)->prepare_srv(newsrv);
}

4
src/server.c
View File

@ -2049,6 +2049,10 @@ static void srv_conn_src_cpy(struct server *srv, struct server *src)
#if defined(USE_OPENSSL)
static void srv_ssl_settings_cpy(struct server *srv, struct server *src)
{
/* <src> is the current proxy's default server and SSL is enabled */
if (src == &srv->proxy->defsrv && src->use_ssl == 1)
srv->flags |= SRV_F_DEFSRV_USE_SSL;
if (src->ssl_ctx.ca_file != NULL)
srv->ssl_ctx.ca_file = strdup(src->ssl_ctx.ca_file);
if (src->ssl_ctx.crl_file != NULL)