RFR: 8297878: KEM: Implementation [v10]

Sean Mullan mullan at openjdk.org
Thu Apr 27 13:28:57 UTC 2023


On Wed, 26 Apr 2023 19:02:53 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.
>
> Weijun Wang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   no more pk/sk, AIOOBE to IOOBE

src/java.base/share/classes/javax/crypto/KEM.java line 90:

> 88: 
> 89:     /**
> 90:      * This class specifies the returned value of the encapsulate method of

s/returned/return/

src/java.base/share/classes/javax/crypto/KEM.java line 100:

> 98:      * @see Encapsulator#encapsulate(int, int, String)
> 99:      */
> 100:     public static final class Encapsulated {

Missing `@since 21`.

src/java.base/share/classes/javax/crypto/KEM.java line 148:

> 146:          * Returns the optional parameters in a byte array.
> 147:          *
> 148:          * @return the optional parameters in a byte array. A new copy of the

Should probably say "the optional parameters in a byte array or `null` if not specified`.

src/java.base/share/classes/javax/crypto/KEM.java line 239:

> 237:          * Returns the size of the shared secret.
> 238:          * <p>
> 239:          * This method can be called to find out the length of the share secret

s/share/shared/

src/java.base/share/classes/javax/crypto/KEM.java line 362:

> 360:          * Returns the size of the shared secret.
> 361:          * <p>
> 362:          * This method can be called to find out the length of the share secret

s/share/shared/

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1179113396
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1179115920
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1179119002
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1179122638
PR Review Comment: https://git.openjdk.org/jdk/pull/13256#discussion_r1179127665



More information about the security-dev mailing list