Remove upper limit on the LimitRequestFieldSize directive

MFC: 149269
Reviewed by: stoddard, jerenkrantz, jim

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

diff --git a/CHANGES b/CHANGES
index fa8b1be7e7705e2fa0c8d48b605dbd3207d3af08..97d20834e677fb11b9c4034a6033d0decf367bac 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
 Changes with Apache 2.0.53
 
-) *) Start keeping track of time-taken-to-process-request again for
+  *) Remove compiled-in upper limit on LimitRequestFieldSize.
+     [Bill Stoddard]
+
+  *) Start keeping track of time-taken-to-process-request again for
      mod_status if ExtendedStatus is enabled. [Jim Jagielski]
 
   *) mod_proxy: Handle client-aborted connections correctly.  PR 32443.

diff --git a/server/core.c b/server/core.c
index f9e0857669eadd3e81d5aa20b4e5522a27e8bcac..f4ff2d5502cca844fd014c19bbeb82a3dcc5112d 100644 (file)
--- a/server/core.c
+++ b/server/core.c
@@ -2567,16 +2567,10 @@ static const char *set_limit_req_fieldsize(cmd_parms *cmd, void *dummy,
     lim = atoi(arg);
     if (lim < 0) {
         return apr_pstrcat(cmd->temp_pool, "LimitRequestFieldsize \"", arg,
-                          "\" must be a non-negative integer (0 = no limit)",
+                          "\" must be a non-negative integer",
                           NULL);
     }
 
-    if (lim > DEFAULT_LIMIT_REQUEST_FIELDSIZE) {
-        return apr_psprintf(cmd->temp_pool, "LimitRequestFieldsize \"%s\" "
-                           "must not exceed the precompiled maximum of %d",
-                            arg, DEFAULT_LIMIT_REQUEST_FIELDSIZE);
-    }
-
     cmd->server->limit_req_fieldsize = lim;
     return NULL;
 }