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

Spec states to invoke the premain method in an agent class if it's public but implementation differs

    XMLWordPrintable

Details

    Description

      In the current implementation of sun/instrument/InstrumentationImpl.java, it allows the premain method be non-public which violates the spec. It should be fixed. It will break any existing java agent with a non-public static premain method that should be fixed. So this change should go into JDK 9 EA for external community testing.

      388 // the premain method should not be required to be public,
      389 // make it accessible so we can call it
      390 // Note: The spec says the following:
      391 // The agent class must implement a public static premain method...

      Attachments

        Issue Links

          csr for

          CSR - null JDK-8248189 Spec states to invoke the premain method in an agent class if it's public but implementation differs

          • P3 - Major loss of function.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-5070281 java.lang.instrument - agent's premain class need not be public

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-6932391 Allow java agents be deployed as modules

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Bug - A problem which impairs or prevents the functions of the product. JDK-8260469 6 java/lang/instrument/PremainClass tests timed out

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Enhancement - null JDK-8260707 java/lang/instrument/PremainClass/InheritAgent0100.java times out

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          links to

          Commit Commit openjdk/jdk/c538cd8b

          Review Review openjdk/jdk/1694

          Activity

            People

              sspitsyn Serguei Spitsyn
              mchung Mandy Chung
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: