From 02f3cf19ed803d20aff9294ce7cb732489951ff5 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Tue, 30 Apr 2019 14:08:41 +0200 Subject: [PATCH] CLEANUP: config: Don't alter listener->maxaccept when nbproc is set to 1 This patch only removes a useless calculation on listener->maxaccept when nbproc is set to 1. Indeed, the following formula has no effet in such case: listener->maxaccept = (listener->maxaccept + nbproc - 1) / nbproc; This patch may be backported as far as 1.5. --- src/cfgparse.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/cfgparse.c b/src/cfgparse.c index 9a03a02bb..48d53e9dd 100644 --- a/src/cfgparse.c +++ b/src/cfgparse.c @@ -3728,9 +3728,8 @@ out_uri_auth_compat: * is bound to. Rememeber that maxaccept = -1 must be kept as it is * used to disable the limit. */ - if (listener->maxaccept > 0) { - if (nbproc > 1) - listener->maxaccept = (listener->maxaccept + 1) / 2; + if (listener->maxaccept > 0 && nbproc > 1) { + listener->maxaccept = (listener->maxaccept + 1) / 2; listener->maxaccept = (listener->maxaccept + nbproc - 1) / nbproc; }