Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8028111

XML readers share the same entity expansion counter

    Details

    • Subcomponent:
    • Introduced In Version:
    • Resolved In Build:
      b124
    • Verification:
      Verified

      Backports

        Description

        Issue described here:
        https://forums.oracle.com/thread/2594170
        Oct 23, 2013 11:07 PM by Borislav.Andruschuk
         
        Description:
         
        After upgrading to 1.7.0_45 we have noticed that StAX XMLInputFactory started throwing an exception on reader creation for our simple XML messages. After some investigation, we found that all XML readers share the same XMLSecurityManager and the same XMLLimitAnalyzer with its aggregated total entity expansion counter. It is my understanding that the counter is supposed to be local per reader and not shared across all of them.
         
        The default entity expansion limit is 64000 and if XMLInputFactory tries to create more than 64000 readers It fails because XMLLimitanalyzer total counter is accumulated and at some point exceeds the limit.
         
        OS version:
         
        I assume It should be related to all OS versions, checked only MacOS and RHEL
         
        Development Kit or Runtime version:

        java version "1.7.0_45"
        Java(TM) SE Runtime Environment (build 1.7.0_45-b18)
        Java HotSpot(TM) 64-Bit Server VM (build 24.45-b08, mixed mode)
         
        Regression information:

        Test case defined below works on prior JDK versions to 1.7.0_45

        Steps to Reproduce:

        1. create XMLInputFactory
         
        XMLInputFactory factory = XMLInputFactory.newInstance();
         
        2. create XMLEventReader:
         
        factory.createXMLEventReader(stream);
         
        3. repeat step 2 64001 times (default entity expansion limit is 64000)
         
        Expected Result:

        XMLInputFactory should be able to create as many XMLReaders as requested
         
        Actual Result:

        JAXP00010001 error

        Error Message(s):
         
        Exception in thread "main" javax.xml.stream.XMLStreamException: ParseError at [row,col]:[1,1]
        Message: JAXP00010001: The parser has encountered more than "64000" entity expansions in this document; this is the limit imposed by the JDK.
        at com.sun.org.apache.xerces.internal.impl.XMLStreamReaderImpl.setInputSource(XMLStreamReaderImpl.java:219)
        at com.sun.org.apache.xerces.internal.impl.XMLStreamReaderImpl.<init>(XMLStreamReaderImpl.java:189)
        at com.sun.xml.internal.stream.XMLInputFactoryImpl.getXMLStreamReaderImpl(XMLInputFactoryImpl.java:277)
        at com.sun.xml.internal.stream.XMLInputFactoryImpl.createXMLStreamReader(XMLInputFactoryImpl.java:129)
        at com.sun.xml.internal.stream.XMLInputFactoryImpl.createXMLEventReader(XMLInputFactoryImpl.java:78)
        at Main.main(Main.java:38)
         
        Source code for an executable test case:

        import java.io.ByteArrayInputStream;
         
        import javax.xml.stream.XMLInputFactory;
         
        public class Main {
         
            public static void main(String[] args) throws Exception {
         
                String xml = "<?xml version=\"1.0\"?><test></test>";
         
                XMLInputFactory factory = XMLInputFactory.newInstance();
         
                for (int i = 0; i < 64001; i++) {
         
                    ByteArrayInputStream stream = new ByteArrayInputStream(xml.getBytes());
                    factory.createXMLEventReader(stream);
                }
            }
        }
         
        Workaround:
         
        The workaround is to disable FPS or entity expansion limit (be careful now your code can be vulnerable to denial of service attack, see replies on Bug ID: JDK-4843787 org.xml.sax.SAXException was thrown when parsing large file )
         
        - set system property jdk.xml.entityExpansionLimit to 0
        - disable FPS in XMLSecurityManager for XMLInputFactory:
         
                XMLInputFactory factory = XMLInputFactory.newInstance();
                XMLSecurityManager securityManager = new XMLSecurityManager(false);
                factory.setProperty("http://apache.org/xml/properties/security-manager", securityManager);

          Issue Links

            Activity

            rcalnan Roger Calnan created issue -
            rcalnan Roger Calnan made changes -
            Field Original Value New Value
            Link This issue relates to JDK-8014530 [ JDK-8014530 ]
            rlewis Roger Lewis made changes -
            Labels regression
            bchristi Brent Christian made changes -
            Status New [ 10000 ] Open [ 1 ]
            bchristi Brent Christian made changes -
            Assignee Joe Wang [ joehw ]
            joehw Joe Wang made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            Understanding Fix Understood [ 10001 ]
            joehw Joe Wang made changes -
            Fix Version/s 8 [ 11815 ]
            Affects Version/s 6u65 [ 15222 ]
            Affects Version/s 5.0u55 [ 15213 ]
            Affects Version/s 8 [ 11815 ]
            joehw Joe Wang made changes -
            Priority P3 [ 3 ] P2 [ 2 ]
            joehw Joe Wang made changes -
            Labels regression 8-critical-request regression
            tyao Ting-Yun Ingrid Yao (Inactive) made changes -
            Labels 8-critical-request regression 8-critical-request CAP cap-8 regression
            jeff Jeff Dinkins made changes -
            Labels 8-critical-request CAP cap-8 regression CAP CPU14_01-critical-request cap-8 regression
            coffeys Sean Coffey made changes -
            Link This issue backported by JDK-8028703 [ JDK-8028703 ]
            coffeys Sean Coffey made changes -
            Link This issue backported by JDK-8028704 [ JDK-8028704 ]
            coffeys Sean Coffey made changes -
            Link This issue backported by JDK-8028705 [ JDK-8028705 ]
            jeff Jeff Dinkins made changes -
            Labels CAP CPU14_01-critical-request cap-8 regression CAP CPU14_01-critical-watch cap-8 regression
            coffeys Sean Coffey made changes -
            Link This issue relates to JDK-8029038 [ JDK-8029038 ]
            joehw Joe Wang made changes -
            Link This issue relates to JDK-6315411 [ JDK-6315411 ]
            joehw Joe Wang made changes -
            Due Date 2013-11-29
            hgupdate HG Updates made changes -
            Understanding Fix Understood [ 10001 ]
            hgupdate HG Updates made changes -
            Status In Progress [ 3 ] Resolved [ 5 ]
            Resolved In Build team [ 17324 ]
            Resolution Fixed [ 1 ]
            joehw Joe Wang made changes -
            Labels CAP CPU14_01-critical-watch cap-8 regression CAP CPU14_01-critical-request cap-8 regression
            shnarasi Shobana Narasimhan made changes -
            Labels CAP CPU14_01-critical-request cap-8 regression CAP CPU14_01-critical-approved cap-8 regression
            asaha Abhijit Saha made changes -
            Labels CAP CPU14_01-critical-approved cap-8 regression CAP CPU14_01-critical-approved cap-8 regression regression_8014530
            asaha Abhijit Saha made changes -
            Link This issue backported by JDK-8029402 [ JDK-8029402 ]
            asaha Abhijit Saha made changes -
            Link This issue backported by JDK-8029403 [ JDK-8029403 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8029404 [ JDK-8029404 ]
            asaha Abhijit Saha made changes -
            Labels CAP CPU14_01-critical-approved cap-8 regression regression_8014530 CAP CPU14_01-critical-approved bugdb_17860263 cap-8 regression regression_8014530
            pzhang Patrick Zhang (Inactive) made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            Verification Verified [ 17000 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8031843 [ JDK-8031843 ]
            hgupdate HG Updates made changes -
            Resolved In Build team [ 17324 ] master [ 18256 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8032124 [ JDK-8032124 ]
            hgupdate HG Updates made changes -
            Resolved In Build master [ 18256 ] b124 [ 17603 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8032336 [ JDK-8032336 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8033825 [ JDK-8033825 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8037759 [ JDK-8037759 ]
            hgupdate HG Updates made changes -
            Link This issue backported by JDK-8039785 [ JDK-8039785 ]
            darcy Joe Darcy made changes -
            Link This issue csr of CCC-8028111 [ CCC-8028111 ]

              People

              • Assignee:
                joehw Joe Wang
                Reporter:
                rcalnan Roger Calnan
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved: