Merge pull request #21 from cesarhernandezgt/tomcat-7.0.109-TT-patch

Prepare for 7.0.109-TT.8 release
tomitribe · Jun 14, 2023 · d0fb908 · d0fb908
2 parents 902657e + fab0295
commit d0fb908
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/build.properties.default b/build.properties.default
@@ -27,7 +27,7 @@ version.major=7
 version.minor=0
 version.build=109
 version.patch=0
-version.suffix=-TT.7
+version.suffix=-TT.8
 
 # ----- Source control flags -----
 git.branch=7.0.x

diff --git a/java/org/apache/tomcat/util/http/Parameters.java b/java/org/apache/tomcat/util/http/Parameters.java
@@ -218,14 +218,14 @@ public void addParameter( String key, String value )
             return;
         }
 
-        parameterCount ++;
-        if (limit > -1 && parameterCount > limit) {
+        if (limit > -1 && parameterCount >= limit) {
             // Processing this parameter will push us over the limit. ISE is
             // what Request.parseParts() uses for requests that are too big
             setParseFailedReason(FailReason.TOO_MANY_PARAMETERS);
             throw new IllegalStateException(sm.getString(
                     "parameters.maxCountFail", Integer.valueOf(limit)));
         }
+        parameterCount++;
 
         ArrayList<String> values = paramHashValues.get(key);
         if (values == null) {