On Mon, 17 Jul 2023 23:06:57 GMT, Valerie Peng <[email protected]> wrote:
>> src/java.base/share/classes/sun/security/rsa/RSASignature.java line 233:
>>
>>> 231: }
>>> 232: return status;
>>> 233: } catch (javax.crypto.BadPaddingException e) {
>>
>> Can this exception still be thrown?
>
> Yes, BadPaddingException is still thrown by RSACore class.
That exception is thrown by RSACore.parseMsg() if the Message is larger than
the modulus. I think it is at least debatable whether it should be a
BadPaddingException. I would rather use an IllegalArgumentException there.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14839#discussion_r1266594911
