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

Proxy::newProxyInstance and MethodHandleProxies::asInterfaceInstance should reject sealed interfaces

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • P2
    • Resolution: Fixed
    • 17
    • 17
    • core-libs
    • None

    Backports

      Description

        Creating as a P2 for now as this may need attention in Java 17.

        Proxy.newProxyInstance doesn't specify how it deals with sealed interfaces. I suspect we may need to update the list of restrictions so that IllegalArgumentException is thrown if an interface is sealed.

        As things stand, a confusing IncompatibleClassChangeError will be thrown when attempting to define the proxy class.

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8270657 Proxy::newProxyInstance and MethodHandleProxies::asInterfaceInstance should reject sealed interfaces

            • P2 - Crashes, loss of data, severe memory leak.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8269463 Proxy::newProxyInstance and MethodHandleProxies::asInterfaceInstance should reject sealed interfaces

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed
            csr for

            CSR - null JDK-8269396 Proxy::newProxyInstance and MethodHandleProxies::asInterfaceInstance should reject sealed interfaces

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed
            links to

            Commit Commit openjdk/jdk17/3d0d27ce

            Review Review openjdk/jdk17/148

            Activity

              People

                mchung Mandy Chung
                alanb Alan Bateman
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: