From: Christopher Faulet Date: Thu, 1 Aug 2024 13:42:09 +0000 (+0200) Subject: BUG/MEDIUM: http-ana: Report error on write error waiting for the response X-Git-Tag: v3.1-dev5~52 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0ba6202796fe24099aeff89a5a4b83af99fc027b;p=thirdparty%2Fhaproxy.git BUG/MEDIUM: http-ana: Report error on write error waiting for the response When we are waiting for the server response, if an error is pending on the frontend side (a write error on client), it is handled as an abort and all regular response analyzers are removed, except the one responsible to release the filters, if any. However, while it is handled as an abort, the error is not reported, as usual, via http_reply_and_close() function. It is an issue because in that, the channels buffers are not reset. Because of this bug, it is possible to block a stream infinitely. The request side is waiting for the response side and the response side is blocked because filters must be released and this cannot be done because data remain blocked in channels buffers. So, in that case, calling http_reply_and_close() with no message is enough to unblock the stream. This patch must be backported as far as 2.8. --- diff --git a/src/http_ana.c b/src/http_ana.c index 3912295239..a62d82cff5 100644 --- a/src/http_ana.c +++ b/src/http_ana.c @@ -1379,6 +1379,7 @@ int http_wait_for_response(struct stream *s, struct channel *rep, int an_bit) http_set_term_flags(s); /* process_stream() will take care of the error */ + http_reply_and_close(s, txn->status, NULL); DBG_TRACE_DEVEL("leaving on error", STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA|STRM_EV_HTTP_ERR, s, txn); return 0;