[nginx] svn commit: r5110 - trunk/src/http

mdounin at mdounin.ru mdounin at mdounin.ru
Tue Mar 12 13:38:05 UTC 2013


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;



More information about the nginx-devel mailing list