This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new cf82828c58 Fix edge case
cf82828c58 is described below
commit cf82828c5895c10ee969503c392676daadcbc15d
Author: remm <[email protected]>
AuthorDate: Wed Dec 11 12:05:34 2024 +0100
Fix edge case
---
java/org/apache/catalina/servlets/DefaultServlet.java | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/java/org/apache/catalina/servlets/DefaultServlet.java
b/java/org/apache/catalina/servlets/DefaultServlet.java
index e066ec417d..76bb988aa5 100644
--- a/java/org/apache/catalina/servlets/DefaultServlet.java
+++ b/java/org/apache/catalina/servlets/DefaultServlet.java
@@ -2282,7 +2282,9 @@ public class DefaultServlet extends HttpServlet {
return true;
}
boolean hasAsteriskValue = false;// check existence of special header
value '*'
+ int headerCount = 0;
while (headerValues.hasMoreElements() && !conditionSatisfied) {
+ headerCount++;
String headerValue = headerValues.nextElement();
if ("*".equals(headerValue)) {
hasAsteriskValue = true;
@@ -2301,7 +2303,11 @@ public class DefaultServlet extends HttpServlet {
}
}
}
- if (hasAsteriskValue && headerValues.hasMoreElements()) {
+ if (headerValues.hasMoreElements()) {
+ headerCount++;
+ }
+
+ if (hasAsteriskValue && headerCount > 1) {
// Note that an If-Match header field with a list value containing
"*" and other values (including other
// instances of "*") is syntactically invalid (therefore not
allowed to be generated) and furthermore is
// unlikely to be interoperable.
@@ -2376,13 +2382,14 @@ public class DefaultServlet extends HttpServlet {
}
boolean hasAsteriskValue = false;// check existence of special header
value '*'
boolean conditionSatisfied = true;
+ int headerCount = 0;
while (headerValues.hasMoreElements()) {
-
+ headerCount++;
String headerValue = headerValues.nextElement();
if (headerValue.equals("*")) {
hasAsteriskValue = true;
- if (headerValues.hasMoreElements()) {
+ if (headerCount > 1 || headerValues.hasMoreElements()) {
conditionSatisfied = false;
break;
} else {
@@ -2422,8 +2429,11 @@ public class DefaultServlet extends HttpServlet {
}
}
+ if (headerValues.hasMoreElements()) {
+ headerCount++;
+ }
- if (hasAsteriskValue && headerValues.hasMoreElements()) {
+ if (hasAsteriskValue && headerCount > 1) {
// Note that an If-None-Match header field with a list value
containing "*" and other values (including
// other instances of "*") is syntactically invalid (therefore not
allowed to be generated) and furthermore
// is unlikely to be interoperable.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
