> 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/ >>> >>> >> >