RFR: 8333364: Minor cleanup could be done in com.sun.crypto.provider
Valerie Peng
valeriep at openjdk.org
Thu Jun 6 20:13:47 UTC 2024
On Tue, 4 Jun 2024 02:32:31 GMT, Mark Powers <mpowers at openjdk.org> wrote:
> https://bugs.openjdk.org/browse/JDK-8333364
src/java.base/share/classes/com/sun/crypto/provider/AESKeyWrap.java line 121:
> 119: @Override
> 120: int encrypt(byte[] pt, int ptOfs, int ptLen, byte[] ct, int ctOfs) {
> 121: throw new UnsupportedOperationException("multi-part not supported");
Change the error message as well?
src/java.base/share/classes/com/sun/crypto/provider/AESKeyWrap.java line 127:
> 125: @Override
> 126: int decrypt(byte[] ct, int ctOfs, int ctLen, byte[] pt, int ptOfs) {
> 127: throw new UnsupportedOperationException("multi-part not supported");
Change the error message as well?
src/java.base/share/classes/com/sun/crypto/provider/AESKeyWrapPadded.java line 159:
> 157: @Override
> 158: int encrypt(byte[] pt, int ptOfs, int ptLen, byte[] ct, int ctOfs) {
> 159: throw new UnsupportedOperationException("multi-part not supported");
Change the error message as well?
src/java.base/share/classes/com/sun/crypto/provider/AESKeyWrapPadded.java line 165:
> 163: @Override
> 164: int decrypt(byte[] ct, int ctOfs, int ctLen, byte[] pt, int ptOfs) {
> 165: throw new UnsupportedOperationException("multi-part not supported");
Change the error message as well?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19535#discussion_r1630167579
PR Review Comment: https://git.openjdk.org/jdk/pull/19535#discussion_r1630167862
PR Review Comment: https://git.openjdk.org/jdk/pull/19535#discussion_r1630169037
PR Review Comment: https://git.openjdk.org/jdk/pull/19535#discussion_r1630169187
More information about the security-dev
mailing list