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:51:27 +0000 (15:51 +0100)
commit406bc7500df823c222134616900657145a5dfdc9
treecb760c5e3aaaa910b8763eb514c452dab28e83fc
parentc2c0e03f1858cc5828f1c55ab94956cb4547c7c8
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.

(cherry picked from commit 0d76a284b6abe90b7001284a5953f8f445c30ebe)
Signed-off-by: Willy Tarreau <w@1wt.eu>
src/h1.c