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
Thu Oct 2 21:09:57 UTC 2025


On Thu, 2 Oct 2025 18:49:53 GMT, Mark Powers <mpowers at openjdk.org> wrote:

>> [JDK-8343232](https://bugs.openjdk.org/browse/JDK-8343232)
>
> 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 364:

> 362:         }
> 363: 
> 364:         if (this.encoded != null)

I don't think it's worth caching the `encoded` inside the object. The `getEncoded` method is only called once for each object. Just return the `encoded` output directly.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2400066890


More information about the security-dev mailing list