Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9d4654c0 authored by Jeff Sharkey's avatar Jeff Sharkey
Browse files

We wanted actual protection level, not flags. 

Bug: 118083154
Test: manual
Change-Id: Ib5b105add2b92401f2dc4a3ca676d3303b46d54f
parent 8d71ec09

services/core/java/com/android/server/pm/permission/DefaultPermissionGrantPolicy.java

+2 −1
Original line number Diff line number Diff line
@@ -1194,6 +1194,7 @@ public final class DefaultPermissionGrantPolicy { 
                } else {

                    permissions.clear();

                }

                permissions.add(permissionGrant.name);

                grantRuntimePermissions(pkg, permissions, permissionGrant.fixed, userId);

            }

        }
@@ -1359,7 +1360,7 @@ public final class DefaultPermissionGrantPolicy { 
    private boolean isPermissionDangerous(String name) {

        try {

            final PermissionInfo pi = mContext.getPackageManager().getPermissionInfo(name, 0);

            return (pi.getProtectionFlags() & PermissionInfo.PROTECTION_DANGEROUS) != 0;

            return (pi.getProtection() == PermissionInfo.PROTECTION_DANGEROUS);

        } catch (NameNotFoundException e) {

            // When unknown assume it's dangerous to be on the safe side

            return true;