]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG: MINOR: http: don't check http-request capture id when len is provided
authorCyril Bonté <cyril.bonte@free.fr>
Thu, 14 Dec 2017 21:44:41 +0000 (22:44 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 14 Dec 2017 21:46:27 +0000 (22:46 +0100)
Randomly, haproxy could fail to start when a "http-request capture"
action is defined, without any change to the configuration. The issue
depends on the memory content, which may raise a fatal error like :
  unable to find capture id 'xxxx' referenced by http-request capture
rule

Commit fd608dd2 already prevents the condition to happen, but this one
should be included for completeness and to reclect the code on the
response side.

The issue was introduced recently by commit 29730ba5 and should only be
backported to haproxy 1.8.

src/proto_http.c

index 1330ac8648d96941d37943edaf9345d09f19550e..481688fdf619defdf76bcc0b272fc0431f789078 100644 (file)
@@ -12149,7 +12149,6 @@ enum act_parse_ret parse_http_req_capture(const char **args, int *orig_arg, stru
 
                rule->action       = ACT_CUSTOM;
                rule->action_ptr   = http_action_req_capture;
-               rule->check_ptr    = check_http_req_capture;
                rule->arg.cap.expr = expr;
                rule->arg.cap.hdr  = hdr;
        }