On Thu, 7 Mar 2024 23:13:14 GMT, Hai-May Chao <hc...@openjdk.org> wrote:

>> Change was made to engineGetEntry() in PKCS12KeyStore to extract the key and 
>> certificate chain from Entry only once. This is because the entry may get 
>> updated between engineGetKey() and engineGetCertificateChain() which causes 
>> inconsistent result. A new test was added to assess and manipulate 
>> PKCS12KeyStore with read and write operations concurrently from multiple 
>> threads. Thanks!
>
> Hai-May Chao has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update engineIsKeyEntry and engineIsCertificateEntry

The delete method also uses alias twice. In fact, the remove method of a map 
already returns the object.

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

PR Comment: https://git.openjdk.org/jdk/pull/18156#issuecomment-1984821539

Reply via email to