Integrated: 8377318: Force bad padding exception in TestPKCS5PaddingError.java at all times
Mikhail Yankelevich
myankelevich at openjdk.org
Tue Feb 17 19:25:19 UTC 2026
On Fri, 6 Feb 2026 16:17:44 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:
> Currently implementation of the test relies on changing the last byte of the cypher to mess up the padding lines. However if the last bit is already 0 the padding will not be taken into account and the result will have no error but will be a garbage output. Current test doesn't take this into account.
>
> The fix should encode the test text without padding and decode with padding to make sure that the bad padding exception is thrown.
>
> Kudos to @djelinski for assisting with finding the issue vs RFC!
>
> Also added a bug id which was missing
This pull request has now been integrated.
Changeset: 4ab05d25
Author: Mikhail Yankelevich <myankelevich at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/4ab05d25c170036cd85155c45e58930fedf614a4
Stats: 19 lines in 1 file changed: 12 ins; 1 del; 6 mod
8377318: Force bad padding exception in TestPKCS5PaddingError.java at all times
Co-authored-by: Daniel Jeliński <djelinski at openjdk.org>
Reviewed-by: valeriep
-------------
PR: https://git.openjdk.org/jdk/pull/29612
More information about the security-dev
mailing list