On Mon, 2 Dec 2024 16:35:55 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Now that JEP 486 has been integrated, `java.security` and `sun.security` >> implementation dependencies on `System.getSecurityManager` and >> `AccessController.doPrivileged` can be removed. >> >> This should cover most of the remaining cleanup in the security libraries >> area. A couple of the `sun.security.action` classes can't be removed yet, as >> well as a few constants in `sun.security.util.SecurityConstants` and >> internal `ProtectionDomain` methods until dependencies are removed in other >> areas. Some additional test cleanup can also be done later. > > Sean Mullan has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains 10 commits: > > - Replace remaining calls to SecurityProperties.privilegedGetOverridable() > with SecurityProperties.getOverridableProperty() and remove > privilegedGetOverridable(). > - Merge > - Remove doPrivileged calls. > - Remove dependency on SecurityConstants.ALL_PERMISSION from java.lang.Class. > - Remove permission text from comments in Provider.java. > - Merge > - Removal of unused fields from AccessControlContext - no longer needed now > that VM cleanup is done. > - More cleanup. > - Merge > - Initial cleanup. I just pushed [a change](https://github.com/openjdk/jdk/pull/22418/commits/a447dd44da5394620c62bac7e05142898d092153) to replace the remaining calls to `SecurityProperties.privilegedGetOverridable()`with `SecurityProperties.getOverridableProperty()` and remove the `privilegedGetOverridable` method. @AlekseiEfimov - this involved one change to `src/java.naming/share/classes/com/sun/naming/internal/ObjectFactoriesFilter.java`. ------------- PR Comment: https://git.openjdk.org/jdk/pull/22418#issuecomment-2512087157