From: Maxim Dounin <mdounin@mdounin.ru>
Date: Mon, 6 Aug 2012 16:24:22 +0000 (+0000)
Subject: Merge of r4758: win32: fixed cpu hog after process startup failure.
X-Git-Tag: release-1.2.3~12
X-Git-Url: http://www.kaiwu.me/postgresql/commit/?a=commitdiff_plain;h=7ca2e6d24e16c55cc336d1c86c9bc315d2199321;p=nginx.git

Merge of r4758: win32: fixed cpu hog after process startup failure.

If ngx_spawn_process() failed while starting a process, the process
handle was closed but left non-NULL in the ngx_processes[] array.
The handle later was used in WaitForMultipleObjects() (if there
were multiple worker processes configured and at least one worker
process was started successfully), resulting in infinite loop.

Reported by Ricardo Villalobos Guevara:
http://mailman.nginx.org/pipermail/nginx-devel/2012-July/002494.html
---

diff --git a/src/os/win32/ngx_process.c b/src/os/win32/ngx_process.c
index 87e8f9a2a..67c6b121a 100644
--- a/src/os/win32/ngx_process.c
+++ b/src/os/win32/ngx_process.c
@@ -196,6 +196,7 @@ failed:
 
     if (ngx_processes[s].handle) {
         ngx_close_handle(ngx_processes[s].handle);
+        ngx_processes[s].handle = NULL;
     }
 
     return NGX_INVALID_PID;