8230415: Avoid redundant permission checking in FilePermissionCollect…

…ion and SocketPermissionCollection Reviewed-by: mullan
openjdk · Sep 28, 2019 · 680fceb · 680fceb
1 parent 68937e9
commit 680fceb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/java.base/share/classes/java/io/FilePermission.java b/src/java.base/share/classes/java/io/FilePermission.java
@@ -1196,7 +1196,7 @@ public boolean implies(Permission permission) {
                 if ((effective & desired) == desired) {
                     return true;
                 }
-                needed = (desired ^ effective);
+                needed = (desired & ~effective);
             }
         }
         return false;

diff --git a/src/java.base/share/classes/java/net/SocketPermission.java b/src/java.base/share/classes/java/net/SocketPermission.java
@@ -1433,7 +1433,7 @@ public boolean implies(Permission permission)
                 if ((effective & desired) == desired) {
                     return true;
                 }
-                needed = (desired ^ effective);
+                needed = (desired & ~effective);
             }
         }
         return false;