On Thu, 14 Nov 2024 14:42:30 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> More obsolete code. Fix trace_class_resolution (doesn't throw exception - >> shouldn't take TRAPS). > > I see a few left over refs to SecurityManager in vmSymbols.hpp, > vmClassMacros.hpp, and a comment in logDiagnosticCommand.hpp. > Thanks @AlanBateman There's a DCmd permissions() function that talks about > DiagnosticCommandMBean security. I don't know what that is so I'm leaving it. > Edit: appears unrelated. Right, no need to change anything there. MBeanServer's spec was changed by JEP 486 to still allow a security exception when access is not authorized. DiagnosticCommandMBean still supports permissions. Kevin Walls is doing a clean-up pass over the java.management and jdk.management to remove vestiges of the security manager but I don't know if he plans to check the VM code. ------------- PR Comment: https://git.openjdk.org/jdk/pull/22064#issuecomment-2478883319
