RFR: 8301553: Support Password-Based Cryptography in SunPKCS11 [v5]
Valerie Peng
valeriep at openjdk.org
Tue May 30 18:07:07 UTC 2023
On Wed, 24 May 2023 19:23:39 GMT, Martin Balao <mbalao at openjdk.org> wrote:
>> 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.
Sure, I think using transient is clearer.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12396#discussion_r1210638477
More information about the security-dev
mailing list