From 948681b8530eaf2c642e1f6e8c78fcae1d90de63 Mon Sep 17 00:00:00 2001 From: Evgeny Kotkov Date: Fri, 7 Jul 2017 11:08:26 +0000 Subject: [PATCH] mpm_winnt: Remove an unnecessary retry after receiving a non-timeout failure from the mpm_get_completion_context() function. Currently, the only possible reasons why mpm_get_completion_context() could fail are real errors such as being unable to WaitForSingleObject(), allocate memory or create an event. Retrying under such circumstances doesn't make sense, and could be as well considered harmful. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1801146 13f79535-47bb-0310-9956-ffa450edef68 --- server/mpm/winnt/child.c | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/server/mpm/winnt/child.c b/server/mpm/winnt/child.c index cf94378f06..c891a1a6f5 100644 --- a/server/mpm/winnt/child.c +++ b/server/mpm/winnt/child.c @@ -438,14 +438,7 @@ reinit: /* target of connect upon too many AcceptEx failures */ context = mpm_get_completion_context(&timeout); if (!context) { if (!timeout) { - /* Hopefully a temporary condition in the provider? */ - ++err_count; - if (err_count > MAX_ACCEPTEX_ERR_COUNT) { - ap_log_error(APLOG_MARK, APLOG_CRIT, 0, ap_server_conf, APLOGNO(00335) - "winnt_accept: Too many failures grabbing a " - "connection ctx. Aborting."); - break; - } + break; } Sleep(100); continue; -- 2.50.1