-
Notifications
You must be signed in to change notification settings - Fork 111
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
Add openssl thread synchronization to fix crashes #283
Conversation
@keithc-ca code updated |
acf9c15
to
1ad5ea6
Compare
ce8de7c
to
475567d
Compare
@keithc-ca Code updated |
475567d
to
ec72993
Compare
@keithc-ca Code updated, I'll update jdk14 and next if jdk11 is ok. |
Jenkins test sanity.openjdk aix,osx,win,xlinux jdk11 |
The xlinux failures appear to be eclipse-openj9/openj9#8899. |
@shanchao95 A gentle reminder that this awaits similar updates to jdk14 and jdknext. |
ec72993
to
b34e17c
Compare
Jenkins test sanity.openjdk aix,osx,win,xlinux jdk11 |
Openssl 1.0.2 is not safely used in multi-thread application. The change adds threadid and locking callback functions to provide thread-safety when openssl 1.0.2 is used. It fixes issue 8373, test crash in openjdk, jdk_net_0, com/sun/net/httpserver/Test9a.java Signed-off-by: Chao Shan <chao.shan@ibm.com>
b34e17c
to
0d5410f
Compare
https://ci.eclipse.org/openj9/view/Pull%20Requests/job/PullRequest-OpenJDK11/92 should still be valid. |
Openssl 1.0.2 is not safely used in multi-thread application.
The change adds threadid and locking callback functions to provide
thread-safety when openssl 1.0.2 is used. It fixes issue 8373,
test crash in openjdk, jdk_net_0,
com/sun/net/httpserver/Test9a.java
Signed-off-by: Chao Shan chao.shan@ibm.com