From b3397367dc7cec9e78c62c54efc24d9db5cde2d2 Mon Sep 17 00:00:00 2001 From: Olivier Houchard Date: Mon, 16 Mar 2020 13:49:00 +0100 Subject: [PATCH] MEDIUM: connections: Kill connections even if we are reusing one. In connect_server(), if we notice we have more file descriptors opened than we should, there's no reason not to close a connection just because we're reusing one, so do it anyway. --- src/backend.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/backend.c b/src/backend.c index cae3f20c5..96d8d17ad 100644 --- a/src/backend.c +++ b/src/backend.c @@ -1256,8 +1256,7 @@ int connect_server(struct stream *s) } } - if (((!reuse || (srv_conn && (srv_conn->flags & CO_FL_WAIT_XPRT))) - && ha_used_fds > global.tune.pool_high_count) && srv && srv->idle_conns) { + if (ha_used_fds > global.tune.pool_high_count && srv && srv->idle_conns) { struct connection *tokill_conn; /* We can't reuse a connection, and e have more FDs than deemd