[Geotools-commits] svn - r38797 - trunk/modules/library/xml/src/main/java/org/geotools/xml

[<svn...@os...>]

Author: aaime
Date: 2012-06-02 01:24:23 -0700 (Sat, 02 Jun 2012)
New Revision: 38797

Modified:
   trunk/modules/library/xml/src/main/java/org/geotools/xml/XMLSAXHandler.java
Log:
Prevent NPE when we have an error but the locator is null (happens when we have invalid charset declaration in the xml

Modified: trunk/modules/library/xml/src/main/java/org/geotools/xml/XMLSAXHandler.java
===================================================================
--- trunk/modules/library/xml/src/main/java/org/geotools/xml/XMLSAXHandler.java	2012-06-02 08:23:48 UTC (rev 38796)
+++ trunk/modules/library/xml/src/main/java/org/geotools/xml/XMLSAXHandler.java	2012-06-02 08:24:23 UTC (rev 38797)
@@ -486,8 +486,10 @@
     public void fatalError(SAXParseException exception)
         throws SAXException {
         logger.severe("FATAL " + exception.getMessage());
-        logger.severe("col " + locator.getColumnNumber() + ", line "
-            + locator.getLineNumber());
+        if(locator != null) {
+            logger.severe("col " + locator.getColumnNumber() + ", line "
+                + locator.getLineNumber());
+        }
         throw exception;
     }