[FIXED JENKINS-15613] Define XML parser property so that Xerces is used.

jenkinsci · Feb 13, 2013 · 00eb7ca · 00eb7ca
1 parent e113d86
commit 00eb7ca
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/pom.xml b/pom.xml
@@ -4,7 +4,7 @@
   <parent>
     <groupId>org.jvnet.hudson.plugins</groupId>
     <artifactId>analysis-pom</artifactId>
-    <version>1.40</version>
+    <version>1.42</version>
     <relativePath>../analysis-pom/pom.xml</relativePath>
   </parent>
 

diff --git a/src/main/java/hudson/plugins/dry/parser/DuplicationParserRegistry.java b/src/main/java/hudson/plugins/dry/parser/DuplicationParserRegistry.java
@@ -16,6 +16,7 @@
 import java.util.Set;
 
 import org.apache.commons.io.IOUtils;
+import org.apache.xerces.parsers.SAXParser;
 
 import com.google.common.collect.Sets;
 
@@ -33,7 +34,10 @@ public class DuplicationParserRegistry implements AnnotationParser {
     private String workspacePath;
     private final String defaultEncoding;
 
-    /**
+    /** Property of SAX parser factory. */
+    private static final String SAX_DRIVER_PROPERTY = "org.xml.sax.driver";
+
+   /**
      * Creates a new instance of {@link DuplicationParserRegistry}.
      *
      * @param highThreshold
@@ -70,6 +74,11 @@ public DuplicationParserRegistry(final int normalThreshold, final int highThresh
 
     /** {@inheritDoc} */
     public Collection<FileAnnotation> parse(final File file, final String moduleName) throws InvocationTargetException {
+        String oldProperty = System.getProperty(SAX_DRIVER_PROPERTY);
+        if (oldProperty != null) {
+            System.setProperty(SAX_DRIVER_PROPERTY, SAXParser.class.getName());
+        }
+
         FileInputStream inputStream = null;
         try {
             for (AbstractDryParser parser : parsers) {
@@ -97,6 +106,9 @@ public Collection<FileAnnotation> parse(final File file, final String moduleName
         }
         finally {
             IOUtils.closeQuietly(inputStream);
+            if (oldProperty != null) {
+                System.setProperty(SAX_DRIVER_PROPERTY, oldProperty);
+            }
         }
     }