ibmruntimes / openj9-openjdk-jdk21

GNU General Public License v2.0
8 stars 40 forks source link

Add additional checks to NativeECDHKeyAgreement to match upstream #157

Closed KostasTsiounis closed 5 months ago

KostasTsiounis commented 6 months ago

There are additional checks that are required during the init and doPhase parts of ECDH key agreement. Those checks are already present in the original ECDHKeyAgreement class, so they had to be added to the NativeECDHKeyAgreement class as well.

Back-ported from: https://github.com/ibmruntimes/openj9-openjdk-jdk/pull/788

Signed-off by: Kostas Tsiounis kostas.tsiounis@ibm.com

KostasTsiounis commented 6 months ago

FYI @keithc-ca

keithc-ca commented 5 months ago

Like https://github.com/ibmruntimes/openj9-openjdk-jdk22/pull/58, I think this should back-port improvements from the head stream, unless you have a compelling reason it shouldn't.

KostasTsiounis commented 5 months ago

Changed to match the head stream.

keithc-ca commented 5 months ago

Jenkins test sanity zlinux jdk21

keithc-ca commented 5 months ago

Grinders passed:

Some tests failed with "java.net.NoRouteToHostException: No route to host" which should be unrelated to this change: