Merge r1766998 from trunk:

Follow up to r1739201.
These APR_TIMEUP special cases are now handled by ap_map_http_request_error().

Reviewed by: jailletc36, jim, covener

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1807753 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index 367beb3451621b38d03a82c0592da49d8d68551c..8c27a5fcd9bf1ad6838e5c471206d5eb4bdc05fc 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -119,13 +119,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
      2.4.x patch: svn merge -c 1801594 ^/httpd/httpd/trunk .
      +1: jfclere, jim, covener
 
-  *) core: Follow up to r1739201.
-           These APR_TIMEUP special cases are now handled by ap_map_http_request_error().
-           (r1739201 has been backported in the 2.4 branch in r1768079)
-     trunk patch: http://svn.apache.org/r1766998
-     2.4.x patch: svn merge -c 1766998 ^/httpd/httpd/trunk .
-     +1: jailletc36, jim, covener
-
   *) buildconf: add an apr-config mode, which allows builds to proceed without
                 APR source code as long as a valid APR installation exists.
                 Involves checking in the latest versions of APR's m4 files and

diff --git a/modules/dav/main/mod_dav.c b/modules/dav/main/mod_dav.c
index 7599c6d8635d28fbbab207819f125ba6161c330c..e3a32466cfbaa2fdf6d1ad549cfe3150aa5e451b 100644 (file)
--- a/modules/dav/main/mod_dav.c
+++ b/modules/dav/main/mod_dav.c
@@ -1025,17 +1025,10 @@ static int dav_method_put(request_rec *r)
             if (rc != APR_SUCCESS) {
                 int http_err;
                 char *msg = ap_escape_html(r->pool, r->uri);
-                if (APR_STATUS_IS_TIMEUP(rc)) {
-                    http_err = HTTP_REQUEST_TIME_OUT;
-                    msg = apr_psprintf(r->pool, "Timeout reading the body "
-                                       "(URI: %s)", msg);
-                }
-                else {
-                    http_err = ap_map_http_request_error(rc, HTTP_BAD_REQUEST);
-                    msg = apr_psprintf(r->pool,
-                            "An error occurred while reading"
-                                    " the request body (URI: %s)", msg);
-                }
+                http_err = ap_map_http_request_error(rc, HTTP_BAD_REQUEST);
+                msg = apr_psprintf(r->pool, "An error occurred while reading "
+                                            "the request body (URI: %s)",
+                                            msg);
                 err = dav_new_error(r->pool, http_err, 0, rc, msg);
                 break;
             }

diff --git a/modules/generators/mod_cgi.c b/modules/generators/mod_cgi.c
index 000f06e5bd52a960560d358bddb08e27864db0d7..8c4a2c63600b6cc0b10fea1333dc2d7d303c49fc 100644 (file)
--- a/modules/generators/mod_cgi.c
+++ b/modules/generators/mod_cgi.c
@@ -853,11 +853,6 @@ static int cgi_handler(request_rec *r)
                             APR_BLOCK_READ, HUGE_STRING_LEN);
 
         if (rv != APR_SUCCESS) {
-            if (APR_STATUS_IS_TIMEUP(rv)) {
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(01224)
-                              "Timeout during reading request entity data");
-                return HTTP_REQUEST_TIME_OUT;
-            }
             ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(01225)
                           "Error reading request entity data");
             return ap_map_http_request_error(rv, HTTP_BAD_REQUEST);

diff --git a/modules/generators/mod_cgid.c b/modules/generators/mod_cgid.c
index 1148467b46d9a671f8ea57c4b637d6efbff719ed..b827ed6ac450f19cedab0d64cbd8ece825415888 100644 (file)
--- a/modules/generators/mod_cgid.c
+++ b/modules/generators/mod_cgid.c
@@ -1534,11 +1534,6 @@ static int cgid_handler(request_rec *r)
                             APR_BLOCK_READ, HUGE_STRING_LEN);
 
         if (rv != APR_SUCCESS) {
-            if (APR_STATUS_IS_TIMEUP(rv)) {
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(01269)
-                              "Timeout during reading request entity data");
-                return HTTP_REQUEST_TIME_OUT;
-            }
             ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r, APLOGNO(01270)
                           "Error reading request entity data");
             return ap_map_http_request_error(rv, HTTP_BAD_REQUEST);