]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: h1: always reject the NUL character in header values
authorWilly Tarreau <w@1wt.eu>
Wed, 31 Jan 2024 14:10:39 +0000 (15:10 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 31 Jan 2024 14:22:12 +0000 (15:22 +0100)
commit0d76a284b6abe90b7001284a5953f8f445c30ebe
treed1a8ca936ef4684b5c3ea433e0066782345a00b3
parent44f02d26f05a953ff28be02b74340cb767cab731
BUG/MEDIUM: h1: always reject the NUL character in header values

Ben Kallus kindly reported that we still hadn't blocked the NUL
character from header values as clarified in RFC9110 and that, even
though there's no known issure related to this, it may one day be
used to construct an attack involving another component.

Actually, both Christopher and I sincerely believed we had done it
prior to releasing 2.9, shame on us for missing that one and thanks
to Ben for the reminder!

The change was applied, it was confirmed to properly reject this NUL
byte from both header and trailer values, and it's still possible to
force it to continue to be supported using the usual pair of unsafe
"option accept-invalid-http-{request|response}" for those who would
like to keep it for whatever reason that wouldn't make sense.

This was tagged medium so that distros also remember to apply it as
a preventive measure.

It should progressively be backported to all versions down to 2.0.
src/h1.c