RFR: 8297878: KEM: Implementation
Sean Mullan
mullan at openjdk.org
Thu Apr 13 17:24:38 UTC 2023
On Fri, 31 Mar 2023 02:25:04 GMT, Weijun Wang <weijun at openjdk.org> wrote:
> The KEM API and DHKEM impl. Note that this PR uses new methods in https://github.com/openjdk/jdk/pull/13250.
src/java.base/share/classes/javax/crypto/KEM.java line 430:
> 428: /**
> 429: * Returns a {@code KEM} object that implements the specified algorithm
> 430: * and supports the specified {@code KEMParameterSpec} parameters
There are no parameters for this method.
src/java.base/share/classes/javax/crypto/KEM.java line 441:
> 439: * to {@link #getInstance(String)}.
> 440: * @return the new {@code KEM} object
> 441: * @throws NoSuchAlgorithmException if no {@code Provider} supports a
This sounds like the provider argument is ignored. Maybe add some more words to take into account both cases "if a {@code provider} is specified and it does not support the specified KEM algorithm, or if {@code provider} is {@code null} and there is no provider that supports a KEM implementation of the specified algorithm"
src/java.base/share/classes/javax/crypto/KEM.java line 460:
> 458: /**
> 459: * Returns a {@code KEM} object that implements the specified algorithm
> 460: * and supports the specified {@code KEMParameterSpec} parameters
There are no parameters.
src/java.base/share/classes/javax/crypto/KEM.java line 471:
> 469: * to {@link #getInstance(String)}.
> 470: * @return the new {@code KEM} object
> 471: * @throws NoSuchAlgorithmException if no {@code Provider} supports a
See above for similar comments.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1165814719
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1165816835
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1165821821
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1165822406
More information about the security-dev
mailing list