BUG/MEDIUM: mux-fcgi: Fix wrong test on FCGI_CF_KEEP_CONN in fcgi_detach()
authorChristopher Faulet <cfaulet@haproxy.com>
Sat, 2 May 2020 07:21:24 +0000 (09:21 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 20 May 2020 16:38:05 +0000 (18:38 +0200)
commite5bcdafdb002d50a2bdcf0df395504cf697bbe2b
treee61453d63f3ee0bca389f540d1fcee8c5400f4ea
parentc00b0b4beef324b64c9b0bff5c510e32a911a54d
BUG/MEDIUM: mux-fcgi: Fix wrong test on FCGI_CF_KEEP_CONN in fcgi_detach()

When a stream is detached from its connection, we try to move the connection in
an idle list to keep it opened, the session one or the server one. But it must
only be done if there is no connection error and if we want to keep it
open. This last statement is true if FCGI_CF_KEEP_CONN flag is set. But the test
is inverted at the stage.

This patch must be backported to 2.1.

(cherry picked from commit 9bcd973a818bcc737f627ef9cef211b33229f18f)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/mux_fcgi.c