]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: http: fix incorrect reporting of server errors
authorWilly Tarreau <w@1wt.eu>
Mon, 2 May 2016 13:25:15 +0000 (15:25 +0200)
committerWilly Tarreau <w@1wt.eu>
Mon, 2 May 2016 14:39:22 +0000 (16:39 +0200)
commitf51d03cf14e68019ab776d4ab2365b15711d85f9
tree81657054ebcea20595649afa4a3a3b8fde55c6dd
parent54e439f0b46f3cb5cd59e1f21864b1790e6d340b
BUG/MEDIUM: http: fix incorrect reporting of server errors

Commit dbe34eb ("MEDIUM: filters/http: Move body parsing of HTTP
messages in dedicated functions") introduced a bug in function
http_response_forward_body() by getting rid of the while(1) loop.
The code immediately following the loop was only reachable on missing
data but now it's also reachable under normal conditions, which used
to be dealt with by the skip_resync_state label returning zero. The
side effect is that in http_server_close situations, the channel's
SHUTR flag is seen and considered as a server error which is reported
if any other error happens (eg: client timeout).

This bug is specific to 1.7, no backport is needed.
src/proto_http.c