RFR: 8073385: Bad error message on parsing illegal character in XML attribute

Aleksej Efimov aleksej.efimov at oracle.com
Mon Mar 30 16:47:41 UTC 2015


Hi Joe,
Sure - I will move the test to jaxp repo and rename it to 
Bug8073385.java (to follow the naming of other test files there).
Thank you for the review.

Thank you,
Aleksej

On 03/30/2015 07:32 PM, huizhe wang wrote:
> Hi Aleksej,
>
> The change looks good.
>
> Could you move the test to the jaxp repo instead of jdk?  You'd find 
> the same package under jaxp/test/javax/xml/jaxp/unittest, and may use 
> any of the javax.xml.parsers tests as an example.
>
> Thanks,
> Joe
>
> On 3/26/2015 9:04 AM, Aleksej Efimov wrote:
>> Hi,
>> Please review the fix for corrupted error messages in XML parser. 
>> When the XML parser encounters illegal
>> character the message with incorrect data is generated:
>> An invalid XML character (Unicode: 0x{2}) was found in the value of 
>> attribute "{1}" and element is "0".
>> But it should be like:
>> An invalid XML character (Unicode: 0x0) was found in the value of 
>> attribute "attTest" and element is "topElement".
>> The fix repairs the message generation and it is similar to the code 
>> in Apache Xerces project.
>> Testing: JTREG and JPRT tests (with new one) showed no failures with 
>> XML related tests.
>>
>> Thanks,
>> Aleksej
>>
>> [1] JBS: https://bugs.openjdk.java.net/browse/JDK-8073385
>> [2] Webrevs:
>>     jaxp: http://cr.openjdk.java.net/~aefimov/8073385/9/00/jaxp/
>>     jdk: http://cr.openjdk.java.net/~aefimov/8073385/9/00/jdk/
>




More information about the core-libs-dev mailing list