RFR: 8302017: Allocate BadPaddingException only if it will be thrown [v2]

Sean Mullan mullan at openjdk.org
Fri Jul 14 20:12:19 UTC 2023


On Wed, 12 Jul 2023 23:12:18 GMT, Valerie Peng <valeriep at 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


More information about the security-dev mailing list