Author: markt Date: Sat Feb 16 22:20:04 2013 New Revision: 1446963 URL: http://svn.apache.org/r1446963 Log: fAllowJavaEncodings is never set so is always false
Modified: tomcat/trunk/java/org/apache/jasper/xmlparser/XMLEncodingDetector.java Modified: tomcat/trunk/java/org/apache/jasper/xmlparser/XMLEncodingDetector.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/xmlparser/XMLEncodingDetector.java?rev=1446963&r1=1446962&r2=1446963&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/jasper/xmlparser/XMLEncodingDetector.java (original) +++ tomcat/trunk/java/org/apache/jasper/xmlparser/XMLEncodingDetector.java Sat Feb 16 22:20:04 2013 @@ -52,7 +52,6 @@ public class XMLEncodingDetector { // org.apache.xerces.impl.XMLEntityManager fields public static final int DEFAULT_BUFFER_SIZE = 2048; public static final int DEFAULT_XMLDECL_BUFFER_SIZE = 64; - private boolean fAllowJavaEncodings; private SymbolTable fSymbolTable; private XMLEncodingDetector fCurrentEntity; private int fBufferSize = DEFAULT_BUFFER_SIZE; @@ -245,8 +244,7 @@ public class XMLEncodingDetector { // check for valid name boolean validIANA = XMLChar.isValidIANAEncoding(encoding); - boolean validJava = XMLChar.isValidJavaEncoding(encoding); - if (!validIANA || (fAllowJavaEncodings && !validJava)) { + if (!validIANA) { err.jspError("jsp.error.xml.encodingDeclInvalid", encoding); // NOTE: AndyH suggested that, on failure, we use ISO Latin 1 // because every byte is a valid ISO Latin 1 character. @@ -262,13 +260,9 @@ public class XMLEncodingDetector { // try to use a Java reader String javaEncoding = EncodingMap.getIANA2JavaMapping(ENCODING); if (javaEncoding == null) { - if (fAllowJavaEncodings) { - javaEncoding = encoding; - } else { - err.jspError("jsp.error.xml.encodingDeclInvalid", encoding); - // see comment above. - javaEncoding = "ISO8859_1"; - } + err.jspError("jsp.error.xml.encodingDeclInvalid", encoding); + // see comment above. + javaEncoding = "ISO8859_1"; } return new InputStreamReader(inputStream, javaEncoding); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org