You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2017/05/02 09:13:40 UTC

svn commit: r1793449 - /tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java

Author: markt
Date: Tue May  2 09:13:40 2017
New Revision: 1793449

URL: http://svn.apache.org/viewvc?rev=1793449&view=rev
Log:
Fix possible NPE

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java?rev=1793449&r1=1793448&r2=1793449&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java Tue May  2 09:13:40 2017
@@ -1124,10 +1124,12 @@ public class Digester extends DefaultHan
         if (locator instanceof Locator2) {
             if (root instanceof DocumentProperties.Charset) {
                 String enc = ((Locator2) locator).getEncoding();
-                try {
-                    ((DocumentProperties.Charset) root).setCharset(B2CConverter.getCharset(enc));
-                } catch (UnsupportedEncodingException e) {
-                    log.warn(sm.getString("disgester.encodingInvalid", enc), e);
+                if (enc != null) {
+                    try {
+                        ((DocumentProperties.Charset) root).setCharset(B2CConverter.getCharset(enc));
+                    } catch (UnsupportedEncodingException e) {
+                        log.warn(sm.getString("disgester.encodingInvalid", enc), e);
+                    }
                 }
             }
         }



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