On Thu, 13 Jul 2023 02:46:10 GMT, Valerie Peng <valer...@openjdk.org> wrote:

> This change adds back the Reference.ReachabilityFence(Object) call removed by 
> [JDK-8301553](https://bugs.openjdk.org/browse/JDK-8301553).
> 
> Please help review.
> Thanks!
> Valerie

src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java line 1:

> 1: /*

I also think the change which moved the registering of the `Cleaner` outside 
the `finally` block in the constructor is not correct, as the passwd is no 
longer zero-ed out if the code after that throws an Exception.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14859#discussion_r1262922031

Reply via email to