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