]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: rules: Initialize the list element when allocating a new rule
authorChristopher Faulet <cfaulet@haproxy.com>
Mon, 21 Mar 2022 06:55:34 +0000 (07:55 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 21 Mar 2022 06:55:37 +0000 (07:55 +0100)
dd7e6c6dc ("BUG/MINOR: http-rules: completely free incorrect TCP rules on
error") and 388c0f2a6 ("BUG/MINOR: tcp-rules: completely free incorrect TCP
rules on error") introduced a regression because the list element of a new
rule is not intialized. Thus HAProxy crashes when an incorrect rule is
released.

This patch must be backported if above commits are backported. Note that
new_act_rule() only exists since the 2.5. It relies on the commit d535f807b
("MINOR: rules: add a new function new_act_rule() to allocate act_rules").

src/action.c

index d4aecc8c0f3044178180eada5880bf328de07299..74f5387b9c5c8a4fe6962abcc2e22a00ec5958d3 100644 (file)
@@ -298,6 +298,7 @@ struct act_rule *new_act_rule(enum act_from from, const char *file, int linenum)
        rule->from = from;
        rule->conf.file = file ? strdup(file) : NULL;
        rule->conf.line = linenum;
+       LIST_INIT(&rule->list);
        return rule;
 }