BAEL-1699 - Java 9 illegal reflective access warning

lsieun · lsieun · commit fe6893e9c98e · 2022-04-07T22:06:01.000+08:00
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A01-compile-and-package-baeldung-agent.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A01-compile-and-package-baeldung-agent.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+DIR=baeldung-agent
+
+# compile
+mkdir -p out/${DIR}
+javac -d out/${DIR} $(find ${DIR} -type f -name "*.java")
+
+# package
+mkdir -p mods
+jar --create --file=mods/${DIR}.jar --manifest=${DIR}/manifest.txt -C out/${DIR} .
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A02-compile-and-package-baeldung-reflected.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A02-compile-and-package-baeldung-reflected.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+DIR=baeldung-reflected
+
+# compile
+mkdir -p out/${DIR}
+javac -d out/${DIR} $(find ${DIR} -type f -name "*.java")
+
+# package
+mkdir -p mods
+jar --create --file=mods/${DIR}.jar -C out/${DIR} .
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A03-compile-and-package-baeldung-intermedium.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A03-compile-and-package-baeldung-intermedium.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+DIR=baeldung-intermedium
+
+# compile
+mkdir -p out/${DIR}
+javac -d out/${DIR} $(find ${DIR} -type f -name "*.java")
+
+# package
+mkdir -p mods
+jar --create --file=mods/${DIR}.jar -C out/${DIR} .
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A04-compile-and-package-baeldung-reflecting-named.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A04-compile-and-package-baeldung-reflecting-named.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+DIR=baeldung-reflecting-named
+
+# compile
+mkdir -p out/${DIR}
+javac -d out/${DIR} --module-path mods $(find ${DIR} -type f -name "*.java")
+
+# package
+mkdir -p mods
+jar --create --file=mods/${DIR}.jar -C out/${DIR} .
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A05-compile-and-package-baeldung-reflecting-unnamed.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/A05-compile-and-package-baeldung-reflecting-unnamed.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+DIR=baeldung-reflecting-unnamed
+
+# compile
+mkdir -p out/${DIR}
+javac -d out/${DIR} $(find ${DIR} -type f -name "*.java")
+
+# package
+mkdir -p mods
+jar --create --file=mods/${DIR}.jar -C out/${DIR} .
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B01-runtime-baeldung-reflecting-named-using-add-opens.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B01-runtime-baeldung-reflecting-named-using-add-opens.sh
@@ -0,0 +1,4 @@
+#!/bin/bash
+java --module-path mods \
+     --add-opens baeldung.reflected/com.baeldung.reflected.internal=baeldung.reflecting.named \
+     --module baeldung.reflecting.named/com.baeldung.reflecting.named.Main
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B02-runtime-baeldung-reflecting-named-using-forward-opens.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B02-runtime-baeldung-reflecting-named-using-forward-opens.sh
@@ -0,0 +1,4 @@
+#!/bin/bash
+java --module-path mods \
+     --add-opens baeldung.reflected/com.baeldung.reflected.internal=baeldung.intermedium \
+     --module baeldung.reflecting.named/com.baeldung.reflecting.named.MainWithForwardOpen
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B03-runtime-baeldung-reflecting-named-using-java-agent.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B03-runtime-baeldung-reflecting-named-using-java-agent.sh
@@ -0,0 +1,4 @@
+#!/bin/bash
+java --module-path mods \
+     -javaagent:mods/baeldung-agent.jar=com.baeldung.reflected.internal.InternalNonPublicClass,com.baeldung.reflecting.named.Main \
+     --module baeldung.reflecting.named/com.baeldung.reflecting.named.Main
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B04-runtime-baeldung-reflecting-unnamed-with-warning.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B04-runtime-baeldung-reflecting-unnamed-with-warning.sh
@@ -0,0 +1,2 @@
+#!/bin/bash
+java -cp "mods/*" com.baeldung.reflecting.unnamed.Main
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B05-runtime-baeldung-reflecting-unnamed-using-add-opens.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B05-runtime-baeldung-reflecting-unnamed-using-add-opens.sh
@@ -0,0 +1,2 @@
+#!/bin/bash
+java -cp "mods/*" --add-opens java.base/java.lang=ALL-UNNAMED com.baeldung.reflecting.unnamed.Main
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B06-runtime-baeldung-reflecting-unnamed-using-java-agent.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/B06-runtime-baeldung-reflecting-unnamed-using-java-agent.sh
@@ -0,0 +1,2 @@
+#!/bin/bash
+java -cp "mods/*" -javaagent:mods/baeldung-agent.jar com.baeldung.reflecting.unnamed.Main
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/C01-clean.sh b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/C01-clean.sh
@@ -0,0 +1,4 @@
+#!/bin/bash
+
+rm -rf out
+rm -rf mods
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/com/baeldung/agent/LoadTimeAgent.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/com/baeldung/agent/LoadTimeAgent.java
@@ -0,0 +1,69 @@
+package com.baeldung.agent;
+
+import java.lang.instrument.Instrumentation;
+import java.util.*;
+
+public class LoadTimeAgent {
+    public static void premain(String agentArgs, Instrumentation inst) {
+        System.out.println("agentArgs: " + agentArgs);
+
+        if (agentArgs != null) {
+            addExportsAndOpensByClassName(inst, agentArgs);
+        }
+        else {
+            addExportsAndOpensToUnnamedModule(inst);
+        }
+    }
+
+    private static void addExportsAndOpensByClassName(Instrumentation inst, String agentArgs) {
+        String[] array = agentArgs.split(",");
+        try {
+            String className1 = array[0];
+            String className2 = array[1];
+            Class<?> clazz1 = Class.forName(className1);
+            Class<?> clazz2 = Class.forName(className2);
+
+            Module srcModule = clazz1.getModule();
+            Module targetModule = clazz2.getModule();
+            redefineModule(inst, srcModule, targetModule);
+        } catch (ClassNotFoundException e) {
+            e.printStackTrace();
+        }
+    }
+
+    private static void addExportsAndOpensToUnnamedModule(Instrumentation inst) {
+        Module unnamedModule = ClassLoader.getSystemClassLoader().getUnnamedModule();
+        Set<Module> modules = ModuleLayer.boot().modules();
+
+        for (Module m : modules) {
+            redefineModule(inst, m, unnamedModule);
+        }
+    }
+
+    private static void redefineModule(Instrumentation inst, Module src, Module target) {
+        // prepare extra reads
+        Set<Module> extraReads = Collections.singleton(target);
+
+        // prepare extra exports
+        Set<String> packages = src.getPackages();
+        Map<String, Set<Module>> extraExports = new HashMap<>();
+        for (String pkg : packages) {
+            extraExports.put(pkg, extraReads);
+        }
+
+        // prepare extra opens
+        Map<String, Set<Module>> extraOpens = new HashMap<>();
+        for (String pkg : packages) {
+            extraOpens.put(pkg, extraReads);
+        }
+
+        // prepare extra uses
+        Set<Class<?>> extraUses = Collections.emptySet();
+
+        // prepare extra uses
+        Map<Class<?>, List<Class<?>>> extraProvides = Collections.emptyMap();
+
+        // redefine module
+        inst.redefineModule(src, extraReads, extraExports, extraOpens, extraUses, extraProvides);
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/manifest.txt b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/manifest.txt
@@ -0,0 +1,3 @@
+Premain-Class: com.baeldung.agent.LoadTimeAgent
+Can-Redefine-Classes: true
+Can-Retransform-Classes: true
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/module-info.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-agent/module-info.java
@@ -0,0 +1,3 @@
+module baeldung.agent {
+    requires java.instrument;
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-intermedium/com/baeldung/intermedium/ForwardOpen.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-intermedium/com/baeldung/intermedium/ForwardOpen.java
@@ -0,0 +1,15 @@
+package com.baeldung.intermedium;
+
+public class ForwardOpen {
+    public static void addOpens(Class<?> clazz1, Class<?> clazz2) {
+        Module currentModule = ForwardOpen.class.getModule();
+        Module srcModule = clazz1.getModule();
+        Module targetModule = clazz2.getModule();
+
+        System.out.println("current module: " + currentModule.getName());
+        System.out.println("source  module: " + srcModule.getName());
+        System.out.println("target  module: " + targetModule.getName());
+
+        srcModule.addOpens(clazz1.getPackageName(), targetModule);
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-intermedium/module-info.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-intermedium/module-info.java
@@ -0,0 +1,3 @@
+module baeldung.intermedium {
+    exports com.baeldung.intermedium;
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/exported/ExportedNonPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/exported/ExportedNonPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.exported;
+
+class ExportedNonPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("ExportedNonPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("ExportedNonPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/exported/ExportedPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/exported/ExportedPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.exported;
+
+public class ExportedPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("ExportedPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("ExportedPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/internal/InternalNonPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/internal/InternalNonPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.internal;
+
+class InternalNonPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("InternalNonPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("InternalNonPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/internal/InternalPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/internal/InternalPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.internal;
+
+public class InternalPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("InternalPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("InternalPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/opened/OpenedNonPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/opened/OpenedNonPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.opened;
+
+class OpenedNonPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("OpenedNonPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("OpenedNonPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/opened/OpenedPublicClass.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/com/baeldung/reflected/opened/OpenedPublicClass.java
@@ -0,0 +1,11 @@
+package com.baeldung.reflected.opened;
+
+public class OpenedPublicClass {
+    public static void testPublicStaticMethod() {
+        System.out.println("OpenedPublicClass.testPublicStaticMethod()");
+    }
+
+    private static void testPrivateStaticMethod() {
+        System.out.println("OpenedPublicClass.testPrivateStaticMethod()");
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/module-info.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflected/module-info.java
@@ -0,0 +1,4 @@
+module baeldung.reflected {
+    exports com.baeldung.reflected.exported;
+    opens com.baeldung.reflected.opened;
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/com/baeldung/reflecting/named/Main.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/com/baeldung/reflecting/named/Main.java
@@ -0,0 +1,12 @@
+package com.baeldung.reflecting.named;
+
+import java.lang.reflect.Method;
+
+public class Main {
+    public static void main(String[] args) throws Exception {
+        Class<?> clazz = Class.forName("com.baeldung.reflected.internal.InternalNonPublicClass");
+        Method method = clazz.getDeclaredMethod("testPrivateStaticMethod");
+        method.setAccessible(true);
+        method.invoke(null);
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/com/baeldung/reflecting/named/MainWithForwardOpen.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/com/baeldung/reflecting/named/MainWithForwardOpen.java
@@ -0,0 +1,18 @@
+package com.baeldung.reflecting.named;
+
+import com.baeldung.intermedium.ForwardOpen;
+
+import java.lang.reflect.Method;
+
+public class MainWithForwardOpen {
+    public static void main(String[] args) throws Exception {
+        Class<?> currentClass = Main.class;
+        Class<?> clazz = Class.forName("com.baeldung.reflected.internal.InternalNonPublicClass");
+
+        ForwardOpen.addOpens(clazz, currentClass);
+
+        Method method = clazz.getDeclaredMethod("testPrivateStaticMethod");
+        method.setAccessible(true);
+        method.invoke(null);
+    }
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/module-info.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-named/module-info.java
@@ -0,0 +1,4 @@
+module baeldung.reflecting.named {
+    requires baeldung.intermedium;
+    requires baeldung.reflected;
+}
diff --git a/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-unnamed/com/baeldung/reflecting/unnamed/Main.java b/core-java-modules/core-java-9-jigsaw/src/modules/com.baeldung.reflective.access/baeldung-reflecting-unnamed/com/baeldung/reflecting/unnamed/Main.java
@@ -0,0 +1,13 @@
+package com.baeldung.reflecting.unnamed;
+
+import java.lang.reflect.Field;
+
+public class Main {
+    public static void main(String[] args) throws Exception {
+        Class<?> clazz = StringBuilder.class;
+        Field f = clazz.getDeclaredField("serialVersionUID");
+        f.setAccessible(true);
+        Object value = f.get(null);
+        System.out.println(value);
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/bin/bash`
	`2`	`+java -cp "mods/*" com.baeldung.reflecting.unnamed.Main`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+#!/bin/bash`
	`2`	`+java -cp "mods/*" --add-opens java.base/java.lang=ALL-UNNAMED com.baeldung.reflecting.unnamed.Main`