-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cloneable test in HmacCore seems questionable #667
Conversation
b0e5257
to
16c4f76
Compare
Hi @keithc-ca , please help to review. |
test/jdk/com/sun/crypto/provider/Mac/DigestCloneabilityTest.java
Outdated
Show resolved
Hide resolved
src/java.base/share/classes/com/sun/crypto/provider/HmacCore.java
Outdated
Show resolved
Hide resolved
src/java.base/share/classes/sun/security/provider/DigestBase.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jinhang Zhang <Jinhang.Zhang@ibm.com>
16c4f76
to
34a3775
Compare
In the future, please ensure commit messages contain enough information that someone reviewing it years down the road will understand the change and why it was made. |
Jenkins test sanity osx jdk11 |
Does this update need to go into the the 0.40 release? |
I think it should (assuming it doesn't cause any issues in the main branch). |
Created #668 for 0.40 |
This is a backport PR from JDK17 PR ibmruntimes/openj9-openjdk-jdk17@b88cc72 to JDK11.