-
Notifications
You must be signed in to change notification settings - Fork 737
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
JDK 24 remove java.security.AccessController.doPrivileged part 2 #21061
Conversation
12cd8fe
to
03ce15d
Compare
Please remove the following from JDK24+ Line 875 in 6ea0111
Can JDK24+ openj9/jcl/src/java.base/share/classes/java/lang/System.java Lines 1387 to 1388 in 64a7502
without throwErrorOnInit ?
|
jcl/src/openj9.cuda/share/classes/com/ibm/cuda/CudaPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.gpu/share/classes/com/ibm/gpu/GPUPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/DumpPermission.java
Outdated
Show resolved
Hide resolved
This method should not throw an error when |
0626094
to
65b95f5
Compare
jcl/src/openj9.cuda/share/classes/com/ibm/cuda/CudaPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/DumpPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/LogPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/ToolDumpPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/ToolDumpPermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/TracePermission.java
Outdated
Show resolved
Hide resolved
jcl/src/openj9.jvm/share/classes/com/ibm/jvm/TracePermission.java
Outdated
Show resolved
Hide resolved
This pull requests is the second of two prs to remove the use of java.security.AccessController.doPrivileged in JDK 24. It also removes a few cases of System.getSystemProperty and SecurityException mentions that were previously missed. Signed-off-by: Theresa Mammarella <[email protected]>
65b95f5
to
a1a3712
Compare
Jenkins test sanity amac jdk21,jdk24 |
This pull requests is the second of two prs to remove the use of java.security.AccessController.doPrivileged in JDK 24. It also removes a few cases of System.getSystemProperty and SecurityException mentions that were previously missed.
Related #20563
@keithc-ca @JasonFengJ9 this is the last of the SecurityManager removal JCL changes for Java 24 that I've found, please let me know if there's any you know of that I've missed.