On my end the agentzh's patch is working correctly.
-Umesh
On 05/17/2012 09:13 AM, agentzh wrote:
> On Thu, May 17, 2012 at 9:06 PM, Maxim Dounin<mdounin@mdounin.ru> wrote:
>> No, disregard this patch. It will break things if new
>> configuration will be rejected for some reason, as global in
>> master will be left in an inconsistent state. As a result
>> incorrect worker processes will be spawn e.g. after abnormal
>> termination of a worker process (or after SIGWINCH + SIGHUP
>> sequence).
>>
>> Allocating temporary copy of a variable looks like the only
>> correct way.
>>
> Indeed. Modifying static/global variables is almost always dangerous
> for configuration reloading :)
>
> Thanks!
> -agentzh
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel