[PATCH]Add processing of posted requests in ngx_http_wait_request_handler().

Maxim Dounin mdounin at mdounin.ru
Tue Mar 18 12:12:30 UTC 2014


Hello!

On Tue, Mar 18, 2014 at 09:58:58AM +0800, flygoast wrote:

> Is this one worked? 

I haven't had time to look into this yet, sorry.

> 
> 
> 
> At 2014-03-07 14:32:43,flygoast <flygoast at 126.com> wrote:
> 
> # HG changeset patch
> # User FengGu <flygoast at 126.com>
> # Date 1394165926 -28800
> #      Fri Mar 07 12:18:46 2014 +0800
> # Node ID c357bd0684560aa2f90f23bdda5dae9cf7f0b13c
> # Parent  e45fa57ef72590d010c8234cb5c3c6e2ae63e157
> Add processing of posted requests in ngx_http_wait_request_handler().
> 
> 
> If something like "error_page 400 /foo" is used in a configuration, when the
> new uri processing failed, the ngx_http_terminate_request(r, -1) would be
> invoked. The request would not be released resulting in socket leak.
> 
> 
> diff -r e45fa57ef725 -r c357bd068456 src/http/ngx_http_request.c
> --- a/src/http/ngx_http_request.c   Thu Mar 06 23:15:10 2014 +0400
> +++ b/src/http/ngx_http_request.c   Fri Mar 07 12:18:46 2014 +0800
> @@ -470,6 +470,7 @@
> 
> 
>      rev->handler = ngx_http_process_request_line;
>      ngx_http_process_request_line(rev);
> +    ngx_http_run_posted_requests(c);
>  }
> 
> 

> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel


-- 
Maxim Dounin
http://nginx.org/



More information about the nginx-devel mailing list