RFR: 8341916: Remove ProtectionDomain related hotspot code and tests [v4]

Alan Bateman alanb at openjdk.org
Fri Nov 15 13:53:53 UTC 2024


On Thu, 14 Nov 2024 14:42:30 GMT, Alan Bateman <alanb at 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


More information about the serviceability-dev mailing list