]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: tcp: tcp rulesets were still broken
authorWilly Tarreau <w@1wt.eu>
Sat, 4 Jul 2015 09:36:30 +0000 (11:36 +0200)
committerWilly Tarreau <w@1wt.eu>
Sat, 4 Jul 2015 09:36:30 +0000 (11:36 +0200)
commit27f78241e6bb3256fafe853e49a0f3e85d8fd933
tree205ec99ae852e2c2b52e6a266c8d56840d8b5c50
parent46175dd81d06d733adbfd4b475bb36fa4273080d
BUG/MAJOR: tcp: tcp rulesets were still broken

Commit cc87a11 ("MEDIUM: tcp: add register keyword system.") broke the
TCP ruleset by merging custom rules and accept. It was fixed a first time
by commit e91ffd0 ("BUG/MAJOR: tcp: only call registered actions when
they're registered") but the accept action still didn't work anymore
and was causing the matching rule to simply be ignored.

Since the code introduced a very fragile behaviour by not even mentionning
that accept and custom were silently merged, let's fix this once for all by
adding an explicit check for the accept action. Nevertheless, as previously
mentionned, the action should be changed so that custom is the only action
and the continue vs break indication directly comes from the callee.

No backport is needed, this bug only affects 1.6-dev.
src/proto_tcp.c