You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by yl...@apache.org on 2018/03/12 15:19:07 UTC
svn commit: r1826555 -
/httpd/httpd/branches/2.4.x/modules/http/http_request.c
Author: ylavic
Date: Mon Mar 12 15:19:07 2018
New Revision: 1826555
URL: http://svn.apache.org/viewvc?rev=1826555&view=rev
Log:
Revert r1826543, was meant for trunk...
Modified:
httpd/httpd/branches/2.4.x/modules/http/http_request.c
Modified: httpd/httpd/branches/2.4.x/modules/http/http_request.c
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/modules/http/http_request.c?rev=1826555&r1=1826554&r2=1826555&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/modules/http/http_request.c (original)
+++ httpd/httpd/branches/2.4.x/modules/http/http_request.c Mon Mar 12 15:19:07 2018
@@ -480,9 +480,13 @@ AP_DECLARE(void) ap_process_request(requ
* Notice a timeout as an error message. This might be
* valuable for detecting clients with broken network
* connections or possible DoS attacks.
+ *
+ * It is still safe to use r / r->pool here as the eor bucket
+ * could not have been destroyed in the event of a timeout.
*/
- ap_log_cerror(APLOG_MARK, APLOG_INFO, rv, c, APLOGNO(01581)
- "flushing data to the client");
+ ap_log_rerror(APLOG_MARK, APLOG_INFO, rv, r, APLOGNO(01581)
+ "Timeout while writing data for URI %s to the"
+ " client", r->unparsed_uri);
}
}
if (ap_extended_status) {