]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: acl: make prue_acl_expr() correctly free ACL expressions upon exit
authorWilly Tarreau <w@1wt.eu>
Fri, 23 Nov 2012 22:53:18 +0000 (23:53 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 23 Nov 2012 23:02:14 +0000 (00:02 +0100)
commit4a6e5c6d69adbd0df9af1dae1a9bce8d42488da1
tree824b37e9fa710881be0edd9e760673dddcb3e0bf
parent7d1df411711ff7aa2943847fdb606a6d1b9966c4
BUG/MEDIUM: acl: make prue_acl_expr() correctly free ACL expressions upon exit

When leaving, during the deinit() process, prune_acl_expr() is called to
delete all ACL expressions. A bug was introduced with commit 34db1084 that
caused every other expression argument to be skipped, and more annoyingly,
it introduced the risk of scanning past the arg list and crashing or
freezing the old process during a reload.

Credits for finding this issue go to Dmitry Sivachenko who first reported
it, and second did a lot of research to narrow it down to a minimal
configuration.
src/acl.c