This site is the archived OWASP Foundation Wiki and is no longer accepting Account Requests.
To view the new OWASP Foundation website, please visit https://owasp.org

Difference between revisions of "Talk:XML External Entity (XXE) Processing"

From OWASP
Jump to: navigation, search
m (Removed non functional code)
Line 14: Line 14:
 
----
 
----
 
<pre>public class SecureEntityResolver implements EntityResolver {
 
<pre>public class SecureEntityResolver implements EntityResolver {
List<String> allowedSystemIds = new ArrayList<String>();
 
  
 
@Override
 
@Override
 
public InputSource resolveEntity(String publicId, String systemId) throws SAXException, IOException {
 
public InputSource resolveEntity(String publicId, String systemId) throws SAXException, IOException {
if(allowedSystemIds.contains(systemId)) {
 
// return a special input source
 
return new InputSource(new ByteArrayInputStream(systemId.getBytes()));
 
}
 
 
return new InputSource(new ByteArrayInputStream("".getBytes()));
 
return new InputSource(new ByteArrayInputStream("".getBytes()));
 
}
 
}
  
public void addAllowedSystemIds(List<String> allowedSystemIds) {
+
 
this.allowedSystemIds = allowedSystemIds;
 
}
 
 
}
 
}
 
</pre>
 
</pre>

Revision as of 15:19, 4 February 2015

XXE Prevention in Java / Using an own EntiyResolver Implementation?



Context: We write Java Software, used in quite different environment (div. Operating System, OpenJDK, IBM-JDK, Sun-JDK, JDK 6 - JDK8). I am not really happy with the described solution to prevent External Entity Injection, because it is depending on some special XML parser Implementations.

To prevent External Entity Injection flaws we primarily use our own EntityResolver and we think this alone should be enough to ensure that the parser can not access resources via URL´s from SGML Entity Declarations.

Our Entity Resolver looks like:


public class SecureEntityResolver implements EntityResolver {

	@Override
	public InputSource resolveEntity(String publicId, String systemId) throws SAXException, IOException {
		return new InputSource(new ByteArrayInputStream("".getBytes()));
	}


}

When parsing XML we register our Entity-Resolver


DocumentBuilderFactory newFactory = DocumentBuilderFactory.newInstance();
newFactory.setNamespaceAware(true);

DocumentBuilder builder = newFactory.newDocumentBuilder();
builder.setEntityResolver(new SecureEntityResolver()); /* !!!! */

Document doc = builder.parse(...);


Does anyone see problems with this way to prevent XXE Injection attacks?

I like your approach in general because it gets away from the silliness of each processor. Have you ran different test cases to see what happens? Even better, could you enable the Java security manager and grant only file access to the test case? I'm interested in seeing if a network connection is attempted during the resolution. --Jon Passki (talk)