Author: markt Date: Sun Mar 17 09:22:16 2013 New Revision: 1457402 URL: http://svn.apache.org/r1457402 Log: There was a reason mark/reset was used here. Restore use of mark/reset and add a comment.
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=1457402&r1=1457401&r2=1457402&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 Sun Mar 17 09:22:16 2013 @@ -364,9 +364,12 @@ public class HttpParser { private static String readTokenOrQuotedString(StringReader input, boolean returnQuoted) throws IOException { + // Use mark/reset as skip(-1) fails when reading the last character of + // the input + input.mark(1); int c = input.read(); - // Skip back so first character is available to be read again - input.skip(-1); + // Go back so first character is available to be read again + input.reset(); if (c == '"') { return readQuotedString(input, returnQuoted); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org