Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8235473

Adjust permission for delayed starting of debugging

XMLWordPrintable

    • Icon: CSR CSR
    • Resolution: Approved
    • Icon: P3 P3
    • 11-pool
    • core-svc
    • None
    • behavioral
    • low
    • A user of the DCmd having access permissions monitor won't be able to enable debugging any longer.
    • Other
    • Implementation

      Summary

      The required permission for DCmd VM.start_java_debugging will be changed from monitor to control.

      Problem

      The DCmd VM.start_java_debugging provides the ability to enable to connect to the VM with a debbugger. As the task of debugging provides the debugger with more than only monitoring capablilities, e.g. fields and data in the VM can be modified, a permission of type "control" seems more suitable. The onjcmd option was introduced with JDK-8214892 and it had been backported to OpenJDK update release 11.0.3. So this permission rectification shall be backported to OpenJDK 11 updates as well to match upstream.

      Solution

      The permission needs to be changed in the coding for diagnostic commands in hotspot.

      Specification

      The (simple) patch looks like this:

      diff --git a/src/hotspot/share/services/diagnosticCommand.hpp b/src/hotspot/share/services/diagnosticCommand.hpp
      --- a/src/hotspot/share/services/diagnosticCommand.hpp
      +++ b/src/hotspot/share/services/diagnosticCommand.hpp
      @@ -880,7 +880,7 @@
      return "High: Switches the VM into Java debug mode.";
      }
      static const JavaPermission permission() {
      - JavaPermission p = { "java.lang.management.ManagementPermission", "monitor", NULL };
      + JavaPermission p = { "java.lang.management.ManagementPermission", "control", NULL };
      return p;
      }
      static int num_arguments() { return 0; }

            clanger Christoph Langer
            rschmelter Ralf Schmelter
            Goetz Lindenmaier, Ralf Schmelter
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: