Merge "Throw a more helpful exception upon invalid split permission configuration." into sc-dev

This commit is contained in:
TreeHugger Robot
2021-04-01 05:37:10 +00:00
committed by Android (Google) Code Review

View File

@@ -3125,6 +3125,10 @@ public class PermissionManagerService extends IPermissionManager.Stub {
if (sourcePerms != null) {
Permission bp = mRegistry.getPermission(newPerm);
if (bp == null) {
throw new IllegalStateException("Unknown new permission in split permission: "
+ newPerm);
}
if (bp.isRuntime()) {
if (!newPerm.equals(Manifest.permission.ACTIVITY_RECOGNITION)) {
@@ -3140,6 +3144,10 @@ public class PermissionManagerService extends IPermissionManager.Stub {
sourcePermNum++) {
final String sourcePerm = sourcePerms.valueAt(sourcePermNum);
Permission sourceBp = mRegistry.getPermission(sourcePerm);
if (sourceBp == null) {
throw new IllegalStateException("Unknown source permission in split"
+ " permission: " + sourcePerm);
}
if (!sourceBp.isRuntime()) {
inheritsFromInstallPerm = true;
break;