Re: RFR(8u): 8072932: Test fails with java.security.AccessControlException: access denied ("java.security.SecurityPermission" "getDomainCombiner")

[Jaroslav Bachorik]

Hi Kevin,

On 3.3.2015 14:29, Kevin Walls wrote:

Hi,

This is a review request for an 8u backport of the bug:
https://bugs.openjdk.java.net/browse/JDK-8072932

It's a one-line change that backports cleanly: this is the 8u diff,
shown after applying the change for 8064331 in 8u: (not hg imported,
different paths)

jdk8u-dev/jdk$ hg diff
src/share/classes/java/security/ProtectionDomain.java
diff --git a/src/share/classes/java/security/ProtectionDomain.java
b/src/share/classes/java/security/ProtectionDomain.java
--- a/src/share/classes/java/security/ProtectionDomain.java
+++ b/src/share/classes/java/security/ProtectionDomain.java
@@ -88,7 +88,7 @@
          }

          private static AccessControlContext
getCombinedACC(AccessControlContext context, AccessControlContext stack) {
-            AccessControlContext acc = new
AccessControlContext(context, stack.getDomainCombiner(), true);
+            AccessControlContext acc = new
AccessControlContext(context, stack.getCombiner(), true);

              return new AccessControlContext(stack.getContext(),
acc).optimize();
          }


Original 9 changeset:
http://hg.openjdk.java.net/jdk9/hs-rt/jdk/rev/f4ce7c7a0296

This looks good. Thanks for taking care of the backports.

-JB-

Thanks
Kevin