Welcome! Log In Create A New Profile

Advanced

[PATCH] Nullify pc->connection in case of failure

Piotr Sikora
January 19, 2012 01:42AM
Hi,
attached patch fixes issue when nginx would try to close the same socket
twice in case of failure to add events on new upstream connection (first
time in ngx_event_connect_peer() and then again in
ngx_http_upstream_finalize_request()).

Best regards,
Piotr Sikora < piotr.sikora@frickle.com >


--- src/event/ngx_event_connect.c.orig Wed Nov 25 18:03:59 2009
+++ src/event/ngx_event_connect.c Thu Jan 19 06:29:09 2012
@@ -247,6 +247,8 @@ failed:
ngx_close_socket_n " failed");
}

+ pc->connection = NULL;
+
return NGX_ERROR;
}

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] Nullify pc->connection in case of failure

Piotr Sikora 1068 January 19, 2012 01:42AM

Re: [PATCH] Nullify pc->connection in case of failure

Maxim Dounin 421 January 19, 2012 10:48AM

Re: [PATCH] Nullify pc->connection in case of failure

Piotr Sikora 439 January 22, 2012 04:42AM

Re: [PATCH] Nullify pc->connection in case of failure

Maxim Dounin 523 January 30, 2012 06:18AM



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

Online Users

Guests: 313
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready