svn commit: r1831760 - /tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java

[markt]

Author: markt
Date: Thu May 17 08:43:32 2018
New Revision: 1831760

URL: http://svn.apache.org/viewvc?rev=1831760&view=rev
Log:
Remove useless condition reported by SpotBugs

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java?rev=1831760&r1=1831759&r2=1831760&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/parser/HttpParser.java Thu 
May 17 08:43:32 2018
@@ -131,7 +131,7 @@ public class HttpParser {
             // Not valid for request target.
             // Combination of multiple rules from RFC7230 and RFC 3986. Must be
             // ASCII, no controls plus a few additional characters excluded
-            if (IS_CONTROL[i] || i > 127 ||
+            if (IS_CONTROL[i] ||
                     i == ' ' || i == '\"' || i == '#' || i == '<' || i == '>' 
|| i == '\\' ||
                     i == '^' || i == '`'  || i == '{' || i == '|' || i == '}') 
{
                 IS_NOT_REQUEST_TARGET[i] = true;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org