Skip to content

Commit 7a4c754

Browse files
committed
8271611: Use SecurityConstants.ACCESS_PERMISSION in MethodHandles
Reviewed-by: rriggs
1 parent e74537f commit 7a4c754

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/java.base/share/classes/java/lang/invoke/MethodHandles.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ public static Lookup privateLookupIn(Class<?> targetClass, Lookup caller) throws
230230

231231
@SuppressWarnings("removal")
232232
SecurityManager sm = System.getSecurityManager();
233-
if (sm != null) sm.checkPermission(ACCESS_PERMISSION);
233+
if (sm != null) sm.checkPermission(SecurityConstants.ACCESS_PERMISSION);
234234
if (targetClass.isPrimitive())
235235
throw new IllegalArgumentException(targetClass + " is a primitive class");
236236
if (targetClass.isArray())
@@ -443,13 +443,10 @@ public static <T> T classDataAt(Lookup caller, String name, Class<T> type, int i
443443
public static <T extends Member> T reflectAs(Class<T> expected, MethodHandle target) {
444444
@SuppressWarnings("removal")
445445
SecurityManager smgr = System.getSecurityManager();
446-
if (smgr != null) smgr.checkPermission(ACCESS_PERMISSION);
446+
if (smgr != null) smgr.checkPermission(SecurityConstants.ACCESS_PERMISSION);
447447
Lookup lookup = Lookup.IMPL_LOOKUP; // use maximally privileged lookup
448448
return lookup.revealDirect(target).reflectAs(expected, lookup);
449449
}
450-
// Copied from AccessibleObject, as used by Method.setAccessible, etc.:
451-
private static final java.security.Permission ACCESS_PERMISSION =
452-
new ReflectPermission("suppressAccessChecks");
453450

454451
/**
455452
* A <em>lookup object</em> is a factory for creating method handles,

0 commit comments

Comments
 (0)