RFR: 8302017: Allocate BadPaddingException only if it will be thrown

Aliaksei Dubrouski duke at openjdk.org
Wed Mar 15 17:57:23 UTC 2023


On Thu, 23 Feb 2023 18:15:35 GMT, Ahmed Muhsin <duke at openjdk.org> wrote:

> This change will move the instantiation of BadPaddingException into the branch of the if statement where it is thrown. This will decrease the overhead of calling `unpadV15` and `unpadOAEP`.  Please see the associated work item for past discussions regarding this change.
> 
> The build and tier1 tests pass locally on mac-aarch64.

I suggested using constructor with disabled stack trace as a backup option, but this will require refactoring few exception classes related to security package (i.e. currently GeneralSecurityException does not have protected constructor with enableSuppression/writableStackTrace flags) and reviewing all usages of them.

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

PR: https://git.openjdk.org/jdk/pull/12732



More information about the security-dev mailing list