Author: mdounin
Date: 2013-03-12 13:38:04 +0000 (Tue, 12 Mar 2013)
New Revision: 5110
URL: http://trac.nginx.org/nginx/changeset/5110/nginx

Log:
Fixed logging in ngx_http_wait_request_handler().

If c->recv() returns 0 there is no sense in using ngx_socket_errno for
logging, its value meaningless. (The code in question was copied from
ngx_http_keepalive_handler(), but ngx_socket_errno makes sense there as it's
used as a part of ECONNRESET handling, and the c->recv() call is preceeded
by the ngx_set_socket_errno(0) call.)


Modified:
trunk/src/http/ngx_http_request.c

Modified: trunk/src/http/ngx_http_request.c
===================================================================
--- trunk/src/http/ngx_http_request.c 2013-03-11 14:44:56 UTC (rev 5109)
+++ trunk/src/http/ngx_http_request.c 2013-03-12 13:38:04 UTC (rev 5110)
@@ -456,7 +456,7 @@
}

if (n == 0) {
- ngx_log_error(NGX_LOG_INFO, c->log, ngx_socket_errno,
+ ngx_log_error(NGX_LOG_INFO, c->log, 0,
"client closed connection");
ngx_http_close_connection(c);
return;

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