]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: cfgparse-tcp: relax namespace bind check
authorDamien Claisse <d.claisse@criteo.com>
Fri, 20 Dec 2024 13:36:34 +0000 (13:36 +0000)
committerWilly Tarreau <w@1wt.eu>
Fri, 7 Mar 2025 15:23:29 +0000 (16:23 +0100)
Commit 5cbb278 introduced cap_sys_admin support, and enforced checks for
both binds and servers. However, when binding into a namespace, the bind
is done before dropping privileges. Hence, checking that we have
cap_sys_admin capability set in this case is not needed (and it would
decrease security to add it).
For users starting haproxy with other user than root and without
cap_sys_admin, bind should have already failed.
As a consequence, relax runtime check for binds into a namespace.

src/cfgparse-tcp.c

index 2c214f3d823ea1600438a3d8878d703c6f4f4d18..e60a18a54f67664491d554f320628e8a00c99888 100644 (file)
@@ -173,7 +173,6 @@ static int bind_parse_namespace(char **args, int cur_arg, struct proxy *px, stru
                ha_alert("Cannot open namespace '%s'.\n", args[cur_arg + 1]);
                return ERR_ALERT | ERR_FATAL;
        }
-       global.last_checks |= LSTCHK_SYSADM;
 
        return 0;
 }