]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: sample: fix case sensitivity for the regsub converter
authorWilly Tarreau <w@1wt.eu>
Fri, 23 Jan 2015 19:23:17 +0000 (20:23 +0100)
committerWilly Tarreau <w@1wt.eu>
Fri, 23 Jan 2015 19:27:41 +0000 (20:27 +0100)
Two commits ago in 7eda849 ("MEDIUM: samples: add a regsub converter to
perform regex-based transformations"), I got caught for the second time
with the inverted case sensitivity usage of regex_comp(). So by default
it is case insensitive and passing the "i" flag makes it case sensitive.
I forgot to recheck that case before committing the cleanup. No harm
anyway, nobody had the time to use it.

src/sample.c

index 598bb533e178b0057f6547620aa6917d598fd27e..a1c6b46060d21649b1b0df5008bc5502f9b41649 100644 (file)
@@ -1201,7 +1201,7 @@ int smp_resolve_args(struct proxy *p)
                        rflags |= (arg->type_flags & ARGF_REG_ICASE) ? REG_ICASE : 0;
                        err = NULL;
 
-                       if (!regex_comp(arg->data.str.str, reg, rflags, 1 /* capture substr */, &err)) {
+                       if (!regex_comp(arg->data.str.str, reg, !(rflags & REG_ICASE), 1 /* capture substr */, &err)) {
                                Alert("parsing [%s:%d] : error in regex '%s' in arg %d of %s%s%s%s '%s' %s proxy '%s' : %s.\n",
                                      cur->file, cur->line,
                                      arg->data.str.str,