RFR: 8278099: two sun/security/pkcs11/Signature tests failed with AssertionError

Daniel D.Daugherty dcubed at openjdk.java.net
Thu Dec 2 15:14:29 UTC 2021


On Thu, 2 Dec 2021 01:20:30 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

> Can someone help reviewing this trivial one-line fix? The assert check in CK_MECHANISM.java is too strict and fail unexpectedly when digest-specific PSS signature mechanisms are supported by the underlying PKCS#11 library. The fix is to remove this assert check. No new regression test added with this fix as this is already covered by existing regression tests.
> 
> Thanks,
> Valerie

I'm not a PKCS11 expert, but the fix does what @valeriepeng describes
in the bug report: removes the offending assert.

Thumbs up.

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

Marked as reviewed by dcubed (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6656


More information about the security-dev mailing list