From 3dae79e59dc34c6e645daa31929410dfa02f01c0 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Thu, 9 Sep 2021 10:17:59 +0200 Subject: [PATCH] BUG/MEDIUM: stream-int: Don't block SI on a channel policy if EOI is reached If the end of input is reported by the mux on the conn-stream during a receive, we leave without evaluating the channel policies. It is especially important to be able to catch client aborts during server connection establishment. Indeed, in this case, without this patch, the stream-interface remains blocked and read events are not forwarded to the stream. It means it is not possible to detect client aborts. Thanks to this fix, the abortonclose option should fixed for HAProxy 2.3 and lower. On 2.4 and 2.5, it seems to work because the stream is created after the request parsing. Note that a previous fix of abortonclose option was reverted. This one should be the right way to fix it. It must carefully be backported as far as 2.0. A observation period on the 2.3 is probably a good idea. (cherry picked from commit 883d83e83c54c77cd16735716a029670b6317926) Signed-off-by: Christopher Faulet (cherry picked from commit 82219b562b512bda3b71489577a23f0c92ea875f) Signed-off-by: Christopher Faulet --- src/stream_interface.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/stream_interface.c b/src/stream_interface.c index 05978a5..69102f7 100644 --- a/src/stream_interface.c +++ b/src/stream_interface.c @@ -1402,6 +1402,14 @@ int si_cs_recv(struct conn_stream *cs) ic->flags |= CF_READ_PARTIAL; ic->total += ret; + /* End-of-input reached, we can leave. In this case, it is + * important to break the loop to not block the SI because of + * the channel's policies.This way, we are still able to receive + * shutdowns. + */ + if (cs->flags & CS_FL_EOI) + break; + if ((ic->flags & CF_READ_DONTWAIT) || --read_poll <= 0) { /* we're stopped by the channel's policy */ si_rx_chan_blk(si); -- 1.7.10.4