]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: mux-pt/mux-h1: Release the pipe on connection error on sending path
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 30 Jul 2024 06:41:03 +0000 (08:41 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 30 Jul 2024 07:05:25 +0000 (09:05 +0200)
commit760d26a8625f3af2b6939037a40f19b5f8063be1
tree6d9840692d507edbdca660576711c66bb7ca5221
parent5dc45445ff18207dbacebf1f777e1f1abcd5065d
BUG/MEDIUM: mux-pt/mux-h1: Release the pipe on connection error on sending path

When data are sent using the kernel splicing, if a connection error
occurred, the pipe must be released. Indeed, in that case, no more data can
be sent and there is no reason to not release the pipe. But it is in fact an
issue for the stream because the channel will appear are not empty. This may
prevent the stream to be released. This happens on 2.8 when a filter is also
attached on it. On 2.9 and upper, it seems there is not issue. But it is
hard to be sure and the current patch remains valid is all cases. On 2.6 and
lower, the code is not the same and, AFAIK, there is no issue.

This patch must be backported to 2.8. However, on 2.8, there is no zero-copy
data forwarding. The patch must be adapted. There is no done_ff/resume_ff
callback functions for muxes. The pipe must released in sc_conn_send() when
an error flag is set on the SE, after the call to snd_pipe callback
function.
src/mux_h1.c
src/mux_pt.c