Code review request, JKD-8024068 sun/security/ssl/javax/net/ssl/ServerName/IllegalSNIName.java fails
Bradford Wetmore
bradford.wetmore at oracle.com
Mon Sep 2 03:02:58 UTC 2013
Same.
Brad
On 9/1/2013 7:45 PM, Weijun Wang wrote:
> Code change looks fine.
>
> --Max
>
> On 9/2/13 10:41 AM, Xuelei Fan wrote:
>> On 9/2/2013 10:38 AM, Xuelei Fan wrote:
>>> Hi Weijun,
>>>
>>> Please review this simple test fix. There is a typo in the test. The
>>> issue is exposed after the fix of JDK-8023881.
>>>
>>> webrev: http://cr.openjdk.java.net/~xuelei/8024068/webrev.00/
>>>
>>> This bug may not have been published to bugs.sun.com. The typo and the
>>> patch look like:
>>>
>>> String[] illegalNames = {
>>> - "example\u3003\u3002com",
>>> + "example\u3002\u3002com",
>>> "example..com",
>>> "com\u3002",
>>> "com.",
>>> "."
>>> };
>>>
>> Note that "\u3002" is a kind of dot code point (ideographic full stop)
>> per RFC 3490.
>>
>> Xuelei
>>
>>> "\u3003" now is acceptable as a Unicode code point, and then result in
>>> test failure.
>>>
>>> Thanks,
>>> Xuelei
>>>
>>
More information about the security-dev
mailing list