> On Aug 24, 2017, at 10:53 PM, Vincent Ryan <vincent.x.r...@oracle.com> wrote:
> 
> 
>> On 24 Aug 2017, at 15:50, Weijun Wang <weijun.w...@oracle.com> wrote:
>> 
>> 1. Maybe just check length > 1?
> 
> Yes. I’ll make the change.
> 
>> 
>> 2. The test's @summary seems unrelated.
> 
> An empty alias is supported in JKS.

Got it.

--Max

> 
> 
>> 
>> --Max
>> 
>>> On Aug 24, 2017, at 10:27 PM, Vincent Ryan <vincent.x.r...@oracle.com> 
>>> wrote:
>>> 
>>> Please review this fix to correct support for an empty string alias in 
>>> PKCS12 keystores.
>>> Thanks.
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8173181
>>> Webrev: http://cr.openjdk.java.net/~vinnie/8173181/webrev.00/
>>> 
>>> 
>> 
> 

Reply via email to