[9] RFR: 8054380: X.509 cert extension SubjectAltName should allow digits as first character of dNSName

Jason Uh jason.uh at oracle.com
Mon Aug 25 20:44:02 UTC 2014



On 08/25/2014 04:14 PM, Sean Mullan wrote:
> On 08/25/2014 04:05 PM, Jason Uh wrote:
>>
>>
>> On 08/25/2014 03:41 PM, Sean Mullan wrote:
>>> Just a couple of comments:
>>>
>>> 1. I think the check on lines 106-110 can be done prior to the for loop
>>> on line 94
>>
>> I think that check is actually where it should be because the for loop
>> on line 94 loops over each component (while the inner loop on line 115
>> checks the characters of that component). The check for the first and
>> last characters is done just once per component.
>
> Right, never mind. There is some extra indentation on line 95 where it
> isn't aligned properly - can you remove the extra spaces?

Thanks for catching that. Corrected here:
http://cr.openjdk.java.net/~juh/8054380/webrev.04/

> --Sean



More information about the security-dev mailing list