commit 9d5f5a0c76dc0aba74777b8afad95e2d6ed12354
author Maxim Dounin <mdounin@mdounin.ru> Mon Jul 23 16:37:24 2012 +0000
committer Maxim Dounin <mdounin@mdounin.ru> Mon Jul 23 16:37:24 2012 +0000
tree c9c1828add30dd321f97d45e61a9e9f867ab6879
parent 54ab41f27bc7e6cb09694c6e6f1fbc9a41034d1a

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 V G:
http://mailman.nginx.org/pipermail/nginx-devel/2012-July/002494.html

src/os/win32/ngx_process.c

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