Welcome! Log In Create A New Profile

Advanced

Re: [PATCH 1 of 3] Upstream: clearing of u->peer.connection on close

October 09, 2011 03:24AM
HI , i want to use 1.1.4 to instead of stable version online ,because we
need the upstream keepalive to do sth, do you recommend it?

2011/9/25 Maxim Dounin <mdounin@mdounin.ru>

> # HG changeset patch
> # User Maxim Dounin <mdounin@mdounin.ru>
> # Date 1316882053 -14400
> # Node ID f1cbffaa09dcc5e8d9f3fc888abcac8e6e7c38b3
> # Parent 6c19b251b92674710573c472e8aba406fa72da50
> Upstream: clearing of u->peer.connection on close.
>
> This fixes crashes observed with some 3rd party balancer modules. Standard
> balancer modules (round-robin and ip hash) explicitly set pc->connection
> (aka u->peer.connection) to NULL and aren't affected.
>
> diff --git a/src/http/ngx_http_upstream.c b/src/http/ngx_http_upstream.c
> --- a/src/http/ngx_http_upstream.c
> +++ b/src/http/ngx_http_upstream.c
> @@ -2927,6 +2927,7 @@ ngx_http_upstream_next(ngx_http_request_
> }
>
> ngx_close_connection(u->peer.connection);
> + u->peer.connection = NULL;
> }
>
> #if 0
>
> _______________________________________________
> 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
Subject Author Views Posted

[PATCH 0 of 3] keepalive/chunked patches

Maxim Dounin 1584 September 25, 2011 06:48AM

[PATCH 1 of 3] Upstream: clearing of u->peer.connection on close

Maxim Dounin 624 September 25, 2011 06:48AM

Re: [PATCH 1 of 3] Upstream: clearing of u->peer.connection on close

xiaohanhoho 577 October 09, 2011 03:24AM

Re: [PATCH 1 of 3] Upstream: clearing of u->peer.connection on close

Maxim Dounin 700 October 09, 2011 03:38PM

[PATCH 2 of 3] Proxy: whitespaces after chunk size allowed

Maxim Dounin 587 September 25, 2011 06:48AM

[PATCH 3 of 3] Proxy: logging levels tuned, double logging fixed

Maxim Dounin 558 September 25, 2011 06:48AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 102
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready