I updated the webrev to also fix the error message that showed the actual number of attributes parsed rather than the limit itself.

http://cr.openjdk.java.net/~joehw/jdk8/8024378/webrev/

Thanks,
Joe

On 10/29/2013 4:49 PM, huizhe wang wrote:
It appears the previous patch was one line short of supporting setting the JAXP's new property ElementAttributeLimit through the StAX factory, that is, resetting the variable in reset(PropertyManager propertyManager) as well. Other than that, I also moved the common reset items from reset(XMLComponentManager...) and reset(PropertyManager ...) to a new method called "resetCommon".

webrevs: http://cr.openjdk.java.net/~joehw/jdk8/8024378/webrev/

The real change is to set the following in both reset methods, now in resetCommon:

663 fElementAttributeLimit = fSecurityManager.getLimit(XMLSecurityManager.Limit.ELEMENT_ATTRIBUTE_LIMIT);


Thanks,
Joe


Reply via email to