BUG/MINOR: checks: Set missing id to the dummy checks frontend
authorChristopher Faulet <cfaulet@haproxy.com>
Fri, 16 Apr 2021 08:49:07 +0000 (10:49 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 21 Apr 2021 08:13:37 +0000 (10:13 +0200)
The dummy frontend used to create the session of the tcp-checks is
initialized without identifier. However, it is required because this id may
be used without any guard, for instance in log-format string via "%f" or
when fe_name sample fetch is called. Thus, an unset id may lead to crashes.

This patch must be backported as far as 2.2.

(cherry picked from commit 0f1fc23d4e3060c4277c11c9f1f41c27d8e54fbf)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>

src/check.c

index 72ee221..9073628 100644 (file)
@@ -1160,6 +1160,7 @@ static int start_checks()
 
        /* 0- init the dummy frontend used to create all checks sessions */
        init_new_proxy(&checks_fe);
+       checks_fe.id = strdup("CHECKS-FE");
        checks_fe.cap = PR_CAP_FE | PR_CAP_BE;
         checks_fe.mode = PR_MODE_TCP;
        checks_fe.maxconn = 0;