RFR: 8301553: Support Password-Based Cryptography in SunPKCS11 [v5]

Martin Balao mbalao at openjdk.org
Wed May 24 19:27:08 UTC 2023


On Wed, 24 May 2023 18:36:34 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Martin Balao has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8301553: Support Password-Based Cryptography in SunPKCS11 (iteration #3)
>>   
>>   Co-authored-by: Francisco Ferrari <fferrari at redhat.com>
>>   Co-authored-by: Martin Balao <mbalao at redhat.com>
>
> src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java line 71:
> 
>> 69:     private Mac prf;
>> 70:     @SuppressWarnings("serial")
>> 71:     private Cleaner.Cleanable cleaner;
> 
> why not directly mark "cleaner" as transient? Then you should not need `@SuppressWarnings` tag.

We thought about transient and then changed to suppress the warning to keep consistency with ```private Mac prf```. I'll change both to transient then.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/12396#discussion_r1204661418



More information about the security-dev mailing list