As reported in 7157610, when setFeature("http://apache.org/xml/features/disallow-doctype-decl";, true) is set on SAXParserFactory object, XMLReader object or DocumentBuilderFactory object, NullPointerException will be thrown when parsing XML document that has DOCTYPE declaration.

The cause of the issue was that when StAX was added, the above feature was reused for a StAX feature, SupportDTD. However, in the StAX spec, the requirement was not always consistent with that of feature 'disallow-doctype-decl'.

The patch is to recover what was the original disallow-doctype-decl, reporting error when disallow-doctype-decl is true, and change everything else that was added for SupportDTD to be governed by a new flag 'fSupportDTD'.

Here's the webrev:
http://cr.openjdk.java.net/~joehw/7u6/7157610/webrev/

All of unit/sqe/tck tests passed.

Please review.

Thanks,
Joe

Reply via email to