RFR: 8343232: PKCS#12 KeyStore support for RFC 9879: Use of Password-Based Message Authentication Code 1 (PBMAC1) [v11]
Weijun Wang
weijun at openjdk.org
Sat Oct 4 18:54:54 UTC 2025
On Fri, 3 Oct 2025 20:53:34 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Mark Powers has updated the pull request incrementally with one additional commit since the last revision:
>>
>> more review comments from Weijun and Sean
>
> src/java.base/share/classes/sun/security/pkcs12/MacData.java line 209:
>
>> 207: }
>> 208: } finally {
>> 209: destroyPBEKey(pbeKey);
>
> The `PBEKeySpec` object created in both cases should also be cleaned up by calling `keySpec.clearPassword()`.
The password in `PBKDF2KeyImpl` is not cleaned. It does have a `clear` method but no `destroy`. Consider adding a `destroy` to it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2404131005
More information about the security-dev
mailing list