]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: stconn: Remove unecessary test on SE_FL_EOS before receiving data
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 21 Mar 2023 09:50:16 +0000 (10:50 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 5 Apr 2023 06:57:05 +0000 (08:57 +0200)
In sc_conn_recv(), if the EOS is reported by the endpoint, it will always be
acknowledged by the SC and a read0 will be performed on the input
channel. Thus there is no reason to still test at the begining of the
function because there is already a test on CF_SHUTR.

src/stconn.c

index c4e26be8652592b20cf72e7dc6f6cab2836ad901..f8dfb7a38d09fa2b9b07d373cf088e46eafdfe5f 100644 (file)
@@ -1196,10 +1196,6 @@ static int sc_conn_recv(struct stconn *sc)
        if (!conn->mux)
                return 0;
 
-       /* stop here if we reached the end of data */
-       if (sc_ep_test(sc, SE_FL_EOS))
-               goto end_recv;
-
        /* stop immediately on errors. Note that we DON'T want to stop on
         * POLL_ERR, as the poller might report a write error while there
         * are still data available in the recv buffer. This typically