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

[Alan Bateman]

On Wed, 4 Dec 2024 13:08:05 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> 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?

The java.management module is mapped to the boot loader so it's okay to replace the use of ReflectUtil.forName with Class.forName here (no behavior change).

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

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