On Thu, 3 Apr 2025 22:58:39 GMT, Mark Powers <[email protected]> wrote:

> [JDK-8343232](https://bugs.openjdk.org/browse/JDK-8343232)

src/java.base/share/classes/com/sun/crypto/provider/PBMAC1Core.java line 168:

> 166:                 } else if (kdfAlgo.equals("HmacSHA256")) {
> 167:                     keyLength = 256;
> 168:                 }

Why only these two algorithms? Doesn't PBMAC1Core also support other HmacSHAXXX 
algorithms? We should add an else block to cover unsupported algorithms to 
detect inconsistency also.

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

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

Reply via email to