From: Christopher Faulet <cfaulet@haproxy.com>
Date: Thu, 21 Apr 2022 10:23:30 +0000 (+0200)
Subject: MINOR: stream: Don't needlessly detach server endpoint on early client abort
X-Git-Tag: v2.6-dev7~13
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c6b60f1e00926cc9db1f20ec56a69d8192a9ff5a;p=thirdparty%2Fhaproxy.git

MINOR: stream: Don't needlessly detach server endpoint on early client abort

When a client abort is detected with the server conn-stream in CS_ST_INI
state, there is no reason to detach the endpoing because we know there is no
endpoint attached to this conn-stream. This patch depends on the commit
"BUG/MEDIUM: conn-stream: Set back CS to RDY state when the appctx is
created".
---

diff --git a/src/stream.c b/src/stream.c
index e428c0c266..ace922d31d 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -2256,7 +2256,6 @@ struct task *process_stream(struct task *t, void *context, unsigned int state)
 			}
 		}
 		else {
-			cs_detach_endp(csb);
 			s->csb->state = CS_ST_CLO; /* shutw+ini = abort */
 			channel_shutw_now(req);        /* fix buffer flags upon abort */
 			channel_shutr_now(res);