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

Kevin Walls kevinw at openjdk.org
Mon Nov 18 17:11:44 UTC 2024


On Mon, 18 Nov 2024 16:07:13 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   add back checkClassLoader
>
> src/java.management/share/classes/com/sun/jmx/remote/security/MBeanServerAccessController.java line 181:
> 
>> 179:         if (sm == null) {
>> 180:             Object object = getMBeanServer().instantiate(className);
>> 181:             checkClassLoader(object);
> 
> I believe we should keep these calls to `checkClassLoader(object)` in place for now. What to do in this case will require some more thinking - I'd suggest handling that part in a separate PR.

Yes agreed.

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

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


More information about the serviceability-dev mailing list