Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Do not get device owner admin info when device owner is null" into nyc-dev am: 21f98459b7

Browse Source 
am: f2ca8f9c50

* commit 'f2ca8f9c508342eae262348f6752bcd53aff2b83':
  Do not get device owner admin info when device owner is null

Change-Id: I29edf9a63d29c8016ee88f9ebe93d3533c09dbed
This commit is contained in:
Ricky Wai
2016-05-31 17:41:07 +00:00
committed by android-build-merger
parent c87bf0ad88 f2ca8f9c50
commit a263edf14f
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java
View File

@@ -3810,12 +3810,16 @@ public class DevicePolicyManagerService extends IDevicePolicyManager.Stub {
// If caller has PO (or DO) it can change the password, so see if that's the case first. // If caller has PO (or DO) it can change the password, so see if that's the case first.
ActiveAdmin admin = getActiveAdminWithPolicyForUidLocked( ActiveAdmin admin = getActiveAdminWithPolicyForUidLocked(
null, DeviceAdminInfo.USES_POLICY_PROFILE_OWNER, callingUid); null, DeviceAdminInfo.USES_POLICY_PROFILE_OWNER, callingUid);
final boolean preN = getTargetSdk(admin.info.getPackageName(), final boolean preN;
if (admin != null) {
preN = getTargetSdk(admin.info.getPackageName(),
userHandle) <= android.os.Build.VERSION_CODES.M; userHandle) <= android.os.Build.VERSION_CODES.M;
if (admin == null) { } else {
// Otherwise, make sure the caller has any active admin with the right policy. // Otherwise, make sure the caller has any active admin with the right policy.
admin = getActiveAdminForCallerLocked(null, admin = getActiveAdminForCallerLocked(null,
DeviceAdminInfo.USES_POLICY_RESET_PASSWORD); DeviceAdminInfo.USES_POLICY_RESET_PASSWORD);
preN = getTargetSdk(admin.info.getPackageName(),
userHandle) <= android.os.Build.VERSION_CODES.M;
// As of N, password resetting to empty/null is not allowed anymore. // As of N, password resetting to empty/null is not allowed anymore.
// TODO Should we allow DO/PO to set an empty password? // TODO Should we allow DO/PO to set an empty password?