]> git.ipfire.org Git - thirdparty/haproxy.git/commit
[BUG] Configuration parser bug when escaping characters
authorCyril Bonté <cyril.bonte@free.fr>
Sun, 6 Dec 2009 12:43:42 +0000 (13:43 +0100)
committerWilly Tarreau <w@1wt.eu>
Sun, 6 Dec 2009 12:46:14 +0000 (13:46 +0100)
commitdd1b01d027bc3f71006d038942c81613b8872edc
treebfbf4310b3eea5f35964aa9b17ce549f9c3dc933
parent68a897b2beb366c777e593eaf022447228ac3808
[BUG] Configuration parser bug when escaping characters

Today I was testing headers manipulation but I met a bug with my first test.
To reproduce it, add for example this line :

    rspadd Cache-Control:\ max-age=1500

Check the response header, it will provide :

Cache-Control: max-age=15000 <= the last character is duplicated

This only happens when we use backslashes on the last line of the
configuration file, without returning to the line.

Also if the last line is like :
  rspadd Cache-Control:\ max-age=1500\

the last backslash causes a segfault.

This is not due to rspadd but to a more general bug in cfgparse.c :
...
if (skip) {
        memmove(line + 1, line + 1 + skip, end - (line + skip + 1));
        end -= skip;
}
...

should be :
...
if (skip) {
        memmove(line + 1, line + 1 + skip, end - (line + skip));
        end -= skip;
}
...

I've reproduced it with haproxy 1.3.22 and the last 1.4 snapshot.
src/cfgparse.c