[XALANJ-2422] TreeWalker set systemId to dummy.xsl always (patch merge issue with XALANJ-1985)

• Type: Bug
• Status: Open
• Priority: Blocker
• Resolution: Unresolved
• Affects Version/s: 2.7.1, 2.7
• Component/s: Xalan
• Assignee: Unassigned
• Reporter: Brian McDonald
• Created: 19/Jan/08 09:47
• Updated: 21/Aug/23 02:13

Description

If you take a look at the merge of the TreeWalker.patch (org/apache/xml/serializer/TreeWalker.java) between versions 338117 and 338053 (for XALANJ-1985), the constructor has some remnant code at the end which breaks it's intended functionality.

This chunk should be removed (sorry for not attaching a patch):

// Set the document locator
if (m_contentHandler != null)
m_contentHandler.setDocumentLocator(m_locator);
try

{ // Bug see Bugzilla 26741 m_locator.setSystemId(System.getProperty("user.dir") + File.separator + "dummy.xsl"); }

catch (SecurityException se){// user.dir not accessible from applet

You can see that the constructor before this chunk already sets the document locator on the content handler and if the system id is null, sets the dummy value. Problem is that this extra chunk is setting the system id no matter what, which breaks all attempts to load relative files, with no workaround possible.

Issue Links

• relates to: XALANJ-1744 org.apache.xml.utils.SystemIDResolver.getAbsoluteURI(String urlString, String base) catches MalformedURIException with applet