From eb601b613dbc909471b2a66c368ae0cace4da8d2 Mon Sep 17 00:00:00 2001 Date: Fri, 22 Jan 2021 11:39:59 +0800 Subject: Backport of JDK-8203699: java/lang/invoke/SpecialInterfaceCall fails with SIGILL on aarch64 summary: Get super_klass value into r0 to make check in VerifyMethodHandles success LLT: jdk/test/java/lang/invoke/lookup/TestDefenderMethodLookup.java Bug url: https://bugs.openjdk.java.net/browse/JDK-8203699 --- .../cpu/aarch64/vm/macroAssembler_aarch64.cpp | 5 + .../lookup/TestDefenderMethodLookup.java | 166 ++++++++++++++++++ 2 files changed, 171 insertions(+) create mode 100644 jdk/test/java/lang/invoke/lookup/TestDefenderMethodLookup.java diff --git a/jdk/test/java/lang/invoke/lookup/TestDefenderMethodLookup.java b/jdk/test/java/lang/invoke/lookup/TestDefenderMethodLookup.java new file mode 100644 index 000000000..8ab268b57 --- /dev/null +++ b/jdk/test/java/lang/invoke/lookup/TestDefenderMethodLookup.java @@ -0,0 +1,164 @@ +/* + * @test + * @bug 8203699 + * @run testng/othervm test.java.lang.invoke.lookup.TestDefenderMethodLookup + */ + +package test.java.lang.invoke.lookup; + +import org.testng.annotations.Test; +import org.testng.Assert; +import java.lang.invoke.*; +import java.lang.invoke.MethodHandle; +import java.lang.invoke.MethodHandles; +import java.lang.invoke.MethodHandles.Lookup; + +//@Test(groups = { "level.sanity" }) +public class TestDefenderMethodLookup { + /** + * Get a SPECIAL MethodHandle for the method "test()V" in the DIRECT super interface DefenderInterface. The method + * has a default implementation in DefenderInterface and does NOT have an implementation in the class. + * Invoke the MethodHandle, and assert that the DefenderInterface.test was invoked (should return "default"). + * + * @throws Throwable No exceptions is expected. Any exception should be treated as an error. + */ + @Test + public void testDirectSuperInterface() throws Throwable { + DefenderInterface impl = new DefenderInterface() { + public MethodHandle run() throws Throwable { + Lookup l = DefenderInterface.lookup(); + Class defc = this.getClass(); + Class target = DefenderInterface.class; + MethodType mt = MethodType.methodType(String.class); + return l.findSpecial(defc, "test", mt, target); + } + }; + MethodHandle mh = impl.run(); + String result = (String)mh.invoke(impl); + Assert.assertEquals("default", result); + } + + /** + * Same as testDirectSuperInterface, but with the findSpecial arguments target and defc switched. + * + * @throws Throwable No exceptions is expected. Any exception should be treated as an error. + */ + @Test + public void testDirectSuperInterfaceSwitchedTargetDefc() throws Throwable { + DefenderInterface impl = new DefenderInterface() { + public MethodHandle run() throws Throwable { + Lookup l = MethodHandles.lookup(); + Class defc = this.getClass(); + Class target = DefenderInterface.class; + MethodType mt = MethodType.methodType(String.class); + // Switched target and defc + return l.findSpecial(target, "test", mt, defc); + } + }; + MethodHandle mh = impl.run(); + String result = (String)mh.invoke(impl); + Assert.assertEquals("default", result); + } + + /** + * Get a SPECIAL MethodHandle for the method "test()V" in the DIRECT super interface DefenderInterface. The method + * has a default implementation in DefenderInterface and does ALSO have an implementation in the class. + * Invoke the MethodHandle, and assert that the DefenderInterface.test was invoked (should return "default"). + * + * @throws Throwable No exceptions is expected. Any exception should be treated as an error. + */ + @Test + public void testDirectSuperInterfaceWithOverride() throws Throwable { + DefenderInterface impl = new DefenderInterface() { + @Test + @Override + public String test() { + return "impl"; + } + + public MethodHandle run() throws Throwable { + Lookup l = DefenderInterface.lookup(); + Class defc = DefenderInterface.class; + Class target = DefenderInterface.class; + MethodType mt = MethodType.methodType(String.class); + return l.findSpecial(defc, "test", mt, target); + } + }; + MethodHandle mh = impl.run(); + String result = (String)mh.invoke(impl); + Assert.assertEquals("default", result); + } + + /** + * Same as testDirectSuperInterfaceWithOverride, but with the findSpecial arguments target and defc switched. + * + * @throws Throwable No exceptions is expected. Any exception should be treated as an error. + */ + @Test + public void testDirectSuperInterfaceWithOverrideSwitchedTargetDefc() throws Throwable { + DefenderInterface impl = new DefenderInterface() { + @Override + public String test() { + return "impl"; + } + + public MethodHandle run() throws Throwable { + Lookup l = MethodHandles.lookup(); + Class defc = this.getClass(); + Class target = DefenderInterface.class; + MethodType mt = MethodType.methodType(String.class); + // Switched target and defc + return l.findSpecial(target, "test", mt, defc); + } + }; + MethodHandle mh = impl.run(); + String result = (String)mh.invoke(impl); + Assert.assertEquals("default", result); + } + + /** + * NEGATIVE
+ * Try to get a SPECIAL MethodHandle for the method "test()V" in the INDIRECT super interface DefenderInterface + * (through the interface DefenderSubInterface). + * + * @throws Throwable Expected exceptions are caught. Any other exception should be treated as an error. + */ + @Test + public void testIndirectSuperInterface() throws Throwable { + DefenderSubInterface impl = new DefenderSubInterface() { + public MethodHandle run() throws Throwable { + Lookup l = DefenderSubInterface.lookup(); + Class defc = this.getClass(); + Class target = DefenderInterface.class; + MethodType mt = MethodType.methodType(String.class); + return l.findSpecial(defc, "test", mt, target); + } + }; + try { + impl.run(); + Assert.fail("Successfully created supersend MethodHandle to INDIRECT super interface. Should fail with IllegalAccessException."); + } catch (IllegalAccessException e) {} + } +} + +interface DefenderInterface { + public default String test() { + return "default"; + } + + public static Lookup lookup() { + return MethodHandles.lookup(); + } + + public MethodHandle run() throws Throwable; +} + +interface DefenderSubInterface extends DefenderInterface { + public default String test() { + return "subDefault"; + } + + public static Lookup lookup() { + return MethodHandles.lookup(); + } +} -- 2.19.0