Yes, I meant \0.

Will fix it.

Thanks
Max

> On Apr 28, 2018, at 1:10 AM, Bernd Eckenfels <e...@zusammenkunft.net> wrote:
> 
> Hello,
> 
> Is the following comment correct, it looks like it should read „with NUL 
> terminator“ instead?
> 
> // without a NULL terminator
> 
> Greetings
> Bernd
> 
> Gruss
> Bernd
> --
> http://bernd.eckenfels.net
> From: security-dev <security-dev-boun...@openjdk.java.net> on behalf of 
> Weijun Wang <weijun.w...@oracle.com>
> Sent: Friday, April 27, 2018 6:56:03 PM
> To: security-dev@openjdk.java.net
> Subject: RFR 8202299: Java Keystore fails to load PKCS12/PFX certificates 
> created in WindowsServer2016
>  
> Please take a look at
> 
>    http://cr.openjdk.java.net/~weijun/8202299/webrev.00/
> 
> Turns out we have to retry [0] other than [] in all 3 locations: decrypting 
> keys, decrypting certs, and verifying the mac.
> 
> Thanks
> Max
> 
> p.s. You might wonder why suddenly in Windows Server 2016, Microsoft starts 
> using [0] to generate the Mac. In fact, they have been doing this all the 
> time. However, before 2016, they also encrypted the certificates, and to 
> decrypt them, Java has already changed password from [] to [0].
> 
> p.p.s. But is this correct? Should the certificate decryption code only 
> temporarily retries [0] but not changing password itself? Well, maybe. But 
> unless a weird software sometimes uses [] and sometimes [0], this will not be 
> a problem, and changing password itself saves us some cycles from always 
> trying twice.

Reply via email to