On Mon, 18 Nov 2024 16:07:13 GMT, Daniel Fuchs <[email protected]> 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
