BUG/MINOR: h1: Report the right error position when a header value is invalid
authorChristopher Faulet <cfaulet@haproxy.com>
Mon, 6 Jan 2020 12:41:01 +0000 (13:41 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 6 Jan 2020 12:58:21 +0000 (13:58 +0100)
commit1703478e2dd6bd12bb03b0a0fdcc7cd4a611dafc
treecf7e5f8684601fad740a9125353b5024855f4261
parent7f4f7f140f6b03b61d1b38260962db235c42c121
BUG/MINOR: h1: Report the right error position when a header value is invalid

During H1 messages parsing, when the parser has finished to parse a full header
line, some tests are performed on its value, depending on its name, to be sure
it is valid. The content-length is checked and converted in integer and the host
header is also checked. If an error occurred during this step, the error
position must point on the header value. But from the parser point of view, we
are already on the start of the next header. Thus the effective reported
position in the error capture is the beginning of the unparsed header line. It
is a bit confusing when we try to figure out why a message is rejected.

Now, the parser state is updated to point on the invalid value. This way, the
error position really points on the right position.

This patch must be backported as far as 1.9.
src/h1.c