RFR: 8344177: Remove SecurityManager and related calls from java.management [v10]

Daniel Fuchs dfuchs at openjdk.org
Wed Dec 4 13:10:42 UTC 2024


On Wed, 4 Dec 2024 10:16:53 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Remove redundant SecurityManager, AccessController references
>> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the Security Manager).
>
> Kevin Walls has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 18 commits:
> 
>  - Merge
>  - Merge remote-tracking branch 'upstream/master' into java_management_post_sm
>  - Exception handling
>  - tidyup
>  - Suppression tidyup
>  - unnecessary variables
>  - remove redundant method
>  - Merge remote-tracking branch 'upstream/master' into java_management_post_sm
>  - ReflectUtil removals
>  - Remove checkPackageAccess calls
>  - ... and 8 more: https://git.openjdk.org/jdk/compare/994504c3...1437532f

src/java.management/share/classes/javax/management/MBeanServerFactory.java line 368:

> 366: 
> 367:         // No context class loader? Try with Class.forName()
> 368:         return Class.forName(builderClassName);

Maybe we should keep that call to ReflectUtil here for the moment?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22100#discussion_r1869463223


More information about the serviceability-dev mailing list