Re: 8244281: test/jdk/com/sun/crypto/provider/KeyProtector/IterationCount.java fails with --illegal-access=deny

[Xuelei Fan]

Looks good to me.

Thanks,
Xuelei

On 5/2/2020 7:30 AM, Alan Bateman wrote:

I need a reviewer for a small update to test/jdk/com/sun/crypto/provider/KeyProtector/IterationCount.java. The test accesses a private field in com.sun.crypto.provider from a VM launched by the test. The launched VM needs to be run with --add-open to open the package, otherwise the test will fail when illegal reflective access is denied by default.

-Alan.

diff -r 104385e30663 test/jdk/com/sun/crypto/provider/KeyProtector/IterationCount.java --- a/test/jdk/com/sun/crypto/provider/KeyProtector/IterationCount.java Sat May 02 09:00:04 2020 +0530 +++ b/test/jdk/com/sun/crypto/provider/KeyProtector/IterationCount.java Sat May 02 15:26:58 2020 +0100

@@ -88,6 +88,7 @@

     private static void executeCommand(List<String> cmd, String expectedCount)

              throws Throwable {
+ cmd.add("--add-opens=java.base/com.sun.crypto.provider=ALL-UNNAMED");
          cmd.add(IterationCount.class.getName());
          cmd.add(clientStr);
          cmd.add(expectedCount);