On Wed, 12 Jul 2023 23:12:18 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> This change refactors the RSAPadding class to return an output record
>> containing the status instead of relying on exception object to indicate a
>> failure.
>>
>> Thanks in advance for review~
>> Valerie
>
> Valerie Peng has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Address review feedbacks, e.g. Removed RSAPadding.Output and use byte[] as
> before.
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?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14839#discussion_r1264109220
