Hello!

On Fri, Feb 13, 2015 at 7:05 AM, Maxim Dounin wrote:
> Rather, I would suggest something like this:
>
> --- a/src/http/ngx_http_upstream.c
> +++ b/src/http/ngx_http_upstream.c
> @@ -3744,10 +3744,13 @@ ngx_http_upstream_finalize_request(ngx_h
> ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
> "finalize http upstream request: %i", rc);
>
> - if (u->cleanup) {
> - *u->cleanup = NULL;
> - u->cleanup = NULL;
> - }
> + if (u->cleanup == NULL) {
> + /* the request was already finalized */
> + ngx_http_finalize_request(r, NGX_DONE);
+ return
> + }
> +
> + *u->cleanup = NULL;
> + u->cleanup = NULL;
>

This patch works for me and yeah it's better. Will you commit it?

Thanks!
-agentzh

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