RFR: 8343232: PKCS#12 KeyStore support for RFC 9579: Use of Password-Based Message Authentication Code 1 (PBMAC1)
Valerie Peng
valeriep at openjdk.org
Tue Sep 2 20:20:46 UTC 2025
On Thu, 3 Apr 2025 22:58:39 GMT, Mark Powers <mpowers at openjdk.org> wrote:
> [JDK-8343232](https://bugs.openjdk.org/browse/JDK-8343232)
src/java.base/share/classes/sun/security/util/PBKDF2Parameters.java line 67:
> 65: * id-hmacWithSHA1 OBJECT IDENTIFIER ::= {digestAlgorithm 7}
> 66: */
> 67: public class PBKDF2Parameters {
nit: can be marked final?
src/java.base/share/classes/sun/security/util/PBKDF2Parameters.java line 72:
> 70: ObjectIdentifier.of(KnownOIDs.PBKDF2WithHmacSHA1);
> 71:
> 72: // AlgorithmIdentifier
nit: comment doesn't seem related or useful? Maybe remove?
src/java.base/share/classes/sun/security/util/PBKDF2Parameters.java line 77:
> 75: private byte[] salt = null;
> 76:
> 77: private int iterationCount = 0;
nit: order the fields matching the ASN.1 definition?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2317061330
PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2317065629
PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2317067786
More information about the security-dev
mailing list