Hi Frederic Well spotted, I will add 'monitor' permission restrictions to this dcmd.
Thanks /F On 2014-01-13 17:40, frederic parain wrote:
The code looks good to me (not an official reviewer). However, I'm surprised this command doesn't require the java.lang.management.ManagementPermission("monitor") when invoked from the DiagnosticCommandMBean. Has this topic been discussed during the CCC review or with the security team? Regards, Fred On 13/01/2014 14:17, Fredrik Arvidsson wrote:Hi Please help me review the following small enhancement: Webrev: http://cr.openjdk.java.net/~farvidsson/8031304/webrev.00/ <http://cr.openjdk.java.net/%7Efarvidsson/8031304/webrev.00/> Jira: https://bugs.openjdk.java.net/browse/JDK-8031304 /Fredrik
