]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MEDIUM: http-ana: Report error on write error waiting for the response
authorChristopher Faulet <cfaulet@haproxy.com>
Thu, 1 Aug 2024 13:42:09 +0000 (15:42 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 2 Aug 2024 06:42:28 +0000 (08:42 +0200)
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.

src/http_ana.c

index 3912295239c9aa14808381751570131c975221f9..a62d82cff50a3223fc489a253ac15955a271144f 100644 (file)
@@ -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;