From: Javen O'Neal <onealj@apache.org>
Date: Thu, 15 Jun 2017 05:00:25 +0000 (+0000)
Subject: bug 61170: add extra logging to SaxHelper.
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f66153c810374d924c0b1f5bccdb44079b1fd876;p=poi.git

bug 61170: add extra logging to SaxHelper.
Thanks to PJ Fanning for the patch!
This closes #58 on github.

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1798792 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/ooxml/java/org/apache/poi/util/SAXHelper.java b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
index 41f14fbcc2..5149a222bb 100644
--- a/src/ooxml/java/org/apache/poi/util/SAXHelper.java
+++ b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
@@ -62,9 +62,20 @@ public final class SAXHelper {
     
     private static final SAXParserFactory saxFactory;
     static {
-        saxFactory = SAXParserFactory.newInstance();
-        saxFactory.setValidating(false);
-        saxFactory.setNamespaceAware(true);
+        try {
+            saxFactory = SAXParserFactory.newInstance();
+            saxFactory.setValidating(false);
+            saxFactory.setNamespaceAware(true);
+        } catch (RuntimeException re) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", re);
+            throw re;
+        } catch (Exception e) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
+            throw new RuntimeException("Failed to create SAXParserFactory", e);
+        } catch (Error e) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
+            throw e;
+        }
     }
             
     private static void trySetSAXFeature(XMLReader xmlReader, String feature) {