On Mon, 25 Oct 2021 17:02:10 GMT, Sean Mullan <mul...@openjdk.org> wrote:

>> You can create a password-less PKCS12 KeyStore file now by calling 
>> `ks.store(outStream, null)` no matter what the default cert protection 
>> algorithm and Mac algorithm are defined in `java.security`.
>> 
>> Note: the system properties set in `ToolsJDK.gmk` to generate `cacerts` must 
>> be retained (at the moment) because the tool is launched with BOOT_JDK.
>
> test/jdk/sun/security/pkcs12/EmptyPassword.java line 57:
> 
>> 55:                 });
>> 56: 
>> 57:         // 8202299: interop before new char[0] and new char[1]
> 
> Can you make this comment more descriptive? Not sure what "before" means. Is 
> this just making sure you can store a keystore and key entry with "\0" as the 
> password and load it back with ""? Should you also try to load it back with 
> "\0" too?

Typo: s/before/between/.

I'll add one with "\0". That should always work since it's the same password 
used in load() and store().

-------------

PR: https://git.openjdk.java.net/jdk/pull/5950

Reply via email to