Merge "DPC should not be allowed to grant development permission" into oc-dev
am: 5e4b2b15ec
Change-Id: I54c0be2c389aacb367b3c8c7e749475dced25f3c
This commit is contained in:
@@ -99,6 +99,7 @@ import android.content.pm.PackageManager;
|
||||
import android.content.pm.PackageManager.NameNotFoundException;
|
||||
import android.content.pm.PackageManagerInternal;
|
||||
import android.content.pm.ParceledListSlice;
|
||||
import android.content.pm.PermissionInfo;
|
||||
import android.content.pm.ResolveInfo;
|
||||
import android.content.pm.ServiceInfo;
|
||||
import android.content.pm.StringParceledListSlice;
|
||||
@@ -152,6 +153,7 @@ import android.telephony.TelephonyManager;
|
||||
import android.text.TextUtils;
|
||||
import android.util.ArrayMap;
|
||||
import android.util.ArraySet;
|
||||
import android.util.EventLog;
|
||||
import android.util.Log;
|
||||
import android.util.Pair;
|
||||
import android.util.Slog;
|
||||
@@ -9582,6 +9584,10 @@ public class DevicePolicyManagerService extends IDevicePolicyManager.Stub {
|
||||
< android.os.Build.VERSION_CODES.M) {
|
||||
return false;
|
||||
}
|
||||
if (!isRuntimePermission(permission)) {
|
||||
EventLog.writeEvent(0x534e4554, "62623498", user.getIdentifier(), "");
|
||||
return false;
|
||||
}
|
||||
final PackageManager packageManager = mInjector.getPackageManager();
|
||||
switch (grantState) {
|
||||
case DevicePolicyManager.PERMISSION_GRANT_STATE_GRANTED: {
|
||||
@@ -9608,6 +9614,8 @@ public class DevicePolicyManagerService extends IDevicePolicyManager.Stub {
|
||||
return true;
|
||||
} catch (SecurityException se) {
|
||||
return false;
|
||||
} catch (NameNotFoundException e) {
|
||||
return false;
|
||||
} finally {
|
||||
mInjector.binderRestoreCallingIdentity(ident);
|
||||
}
|
||||
@@ -9657,6 +9665,13 @@ public class DevicePolicyManagerService extends IDevicePolicyManager.Stub {
|
||||
}
|
||||
}
|
||||
|
||||
public boolean isRuntimePermission(String permissionName) throws NameNotFoundException {
|
||||
final PackageManager packageManager = mInjector.getPackageManager();
|
||||
PermissionInfo permissionInfo = packageManager.getPermissionInfo(permissionName, 0);
|
||||
return (permissionInfo.protectionLevel & PermissionInfo.PROTECTION_MASK_BASE)
|
||||
== PermissionInfo.PROTECTION_DANGEROUS;
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean isProvisioningAllowed(String action, String packageName) {
|
||||
Preconditions.checkNotNull(packageName);
|
||||
|
||||
Reference in New Issue
Block a user