On Tuesday 18 March 2014 09:58:58 flygoast wrote: > Is this one worked? > > > At 2014-03-07 14:32:43,flygoast <flygo...@126.com> wrote: > > # HG changeset patch > # User FengGu <flygo...@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); > } >
Seems to me, this is a wrong patch, since ngx_http_wait_request_handler() isn't the only place, where ngx_http_process_request_line() can be invoked. wbr, Valentin V. Bartenev _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel
