On Tue, 23 Apr 2024 20:42:51 GMT, Kevin Driver <kdri...@openjdk.org> wrote:

> Introduce an API for Key Derivation Functions (KDFs), which are cryptographic 
> algorithms for deriving additional keys from a secret key and other data. See 
> [JEP 478](https://openjdk.org/jeps/478).

src/java.base/share/classes/javax/crypto/KDF.java line 147:

> 145:      *     algorithm.
> 146:      */
> 147:     public static KDF getInstance(String algorithm) throws 
> NoSuchAlgorithmException {

All the `getInstance` methods should throw `NullPointerException` if 
`algorithm` is `null`.

src/java.base/share/classes/javax/crypto/KDF.java line 150:

> 148:         KDF instance = null;
> 149:         try {
> 150:             instance = getInstance(algorithm, (AlgorithmParameterSpec) 
> null);

You can return the instance directly, no need to store it in a local variable.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1595430522
PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1595431991

Reply via email to