]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: http: chunk parser was broken with buffer changes
authorWilly Tarreau <w@1wt.eu>
Thu, 27 Sep 2012 13:08:56 +0000 (15:08 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 27 Sep 2012 13:08:56 +0000 (15:08 +0200)
commitb8ffd378f0dfe57e4b613db80600816e83d8e2fa
tree5f809312db3f52c921ae8ee21d9383800ece1a20
parent3c7a79dbb1ecb35717c7a730bec3546aad1c0ede
BUG/MAJOR: http: chunk parser was broken with buffer changes

Since at least commit a458b679, msg->sov could become negative in
http_parse_chunk_size() if a chunk size wrapped around the buffer.
The effect is that at some point channel_forward() was called with
a negative size, causing all data to be transferred without being
analyzed anymore.

Since haproxy does not support keep-alive with the server yet, this
issue is not really noticeable, as the server closes the connection
in response. Still, when tunnel mode is used or when pretent-keepalive
is used, it is possible to see the problem.

This issue was reported and diagnosed by William Lallemand at
Exceliance.
src/proto_http.c