RFR: 8344397: Remove Security Manager dependencies from java.security and sun.security packages [v2]
Sean Mullan
mullan at openjdk.org
Mon Dec 2 15:32:41 UTC 2024
On Thu, 28 Nov 2024 17:18:36 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Removal of unused fields from AccessControlContext - no longer needed now that VM cleanup is done.
>
> src/java.base/share/classes/sun/security/util/SecurityConstants.java line 61:
>
>> 59: // Permission constants used in the various checkPermission() calls in JDK.
>> 60:
>> 61: // java.lang.Class, java.net.URLConnection, java.security.AllPermission
>
> BTW: It's okay to change the usage in Class.Holder.<clinit> to use `new AllPermission()`. That will help get to the point where this file can be removed.
Fixed in https://github.com/openjdk/jdk/pull/22418/commits/5ad5e8c8b13e3aa8f8379def6b3bb3e994fd084d.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22418#discussion_r1866060865
More information about the security-dev
mailing list