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

Commit 4c308df1 authored by Antoan Angelov's avatar Antoan Angelov Committed by Automerger Merge Worker
Browse files

[RESTRICT AUTOMERGE] Revert "[RESTRICT AUTOMERGE] Remove DPMS#getProfileOwner... 

[RESTRICT AUTOMERGE] Revert "[RESTRICT AUTOMERGE] Remove DPMS#getProfileOwner in favor of getProfileOwnerAsUser" am: c6bda518

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/13462269

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ia1a75f3468b7c316fb75dcfa2e270c73ee7a4553
parents b4d05a90 c6bda518

core/java/android/app/admin/DevicePolicyManager.java

+1 −1
Original line number Diff line number Diff line
@@ -7008,7 +7008,7 @@ public class DevicePolicyManager { 
        throwIfParentInstance("isProfileOwnerApp");

        if (mService != null) {

            try {

                ComponentName profileOwner = mService.getProfileOwnerAsUser(myUserId());

                ComponentName profileOwner = mService.getProfileOwner(myUserId());

                return profileOwner != null

                        && profileOwner.getPackageName().equals(packageName);

            } catch (RemoteException re) {

core/java/android/app/admin/IDevicePolicyManager.aidl

+1 −0
Original line number Diff line number Diff line
@@ -156,6 +156,7 @@ interface IDevicePolicyManager { 


    boolean setProfileOwner(in ComponentName who, String ownerName, int userHandle);

    ComponentName getProfileOwnerAsUser(int userHandle);

    ComponentName getProfileOwner(int userHandle);

    ComponentName getProfileOwnerOrDeviceOwnerSupervisionComponent(in UserHandle userHandle);

    String getProfileOwnerName(int userHandle);

    void setProfileEnabled(in ComponentName who);

services/devicepolicy/java/com/android/server/devicepolicy/CertificateMonitor.java

+1 −1
Original line number Diff line number Diff line
@@ -177,7 +177,7 @@ public class CertificateMonitor { 


        int parentUserId = userHandle.getIdentifier();



        if (mService.getProfileOwnerAsUser(userHandle.getIdentifier()) != null) {

        if (mService.getProfileOwner(userHandle.getIdentifier()) != null) {

            contentText = resources.getString(R.string.ssl_ca_cert_noti_managed,

                    mService.getProfileOwnerName(userHandle.getIdentifier()));

            smallIconId = R.drawable.stat_sys_certificate_info;

services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java

+18 −12
Original line number Diff line number Diff line
@@ -4723,7 +4723,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager { 
    public boolean isSeparateProfileChallengeAllowed(int userHandle) {

        enforceSystemCaller("query separate challenge support");













        ComponentName profileOwner = getProfileOwnerAsUser(userHandle);













        ComponentName profileOwner = getProfileOwner(userHandle);















        // Profile challenge is supported on N or newer release.















        return profileOwner != null &&















                getTargetSdk(profileOwner.getPackageName(), userHandle) > Build.VERSION_CODES.M;










@@ -6638,7 +6638,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {



























        final UserHandle caller = mInjector.binderGetCallingUserHandle();















        // If there is a profile owner, redirect to that; otherwise query the device owner.













        ComponentName aliasChooser = getProfileOwnerAsUser(caller.getIdentifier());













        ComponentName aliasChooser = getProfileOwner(caller.getIdentifier());















        if (aliasChooser == null && caller.isSystem()) {















            synchronized (getLockObject()) {















                final ActiveAdmin deviceOwnerAdmin = getDeviceOwnerAdminLocked();










@@ -8761,7 +8761,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













    }





























    public boolean isProfileOwner(ComponentName who, int userId) {













        final ComponentName profileOwner = getProfileOwnerAsUser(userId);













        final ComponentName profileOwner = getProfileOwner(userId);















        return who != null && who.equals(profileOwner);















    }
























@@ -9319,10 +9319,16 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {



























    @Override















    public ComponentName getProfileOwnerAsUser(int userHandle) {













        enforceCrossUsersPermission(userHandle);

























        return getProfileOwner(userHandle);













    }

























    @Override













    public ComponentName getProfileOwner(int userHandle) {















        if (!mHasFeature) {















            return null;















        }













        enforceCrossUsersPermission(userHandle);















        synchronized (getLockObject()) {















            return mOwners.getProfileOwnerComponent(userHandle);















        }










@@ -9365,9 +9371,9 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













        return mInjector.binderWithCleanCallingIdentity(() -> {















            for (UserInfo userInfo : mUserManager.getProfiles(userHandle)) {















                if (userInfo.isManagedProfile()) {













                    if (getProfileOwnerAsUser(userInfo.id) != null













                    if (getProfileOwner(userInfo.id) != null















                            && isProfileOwnerOfOrganizationOwnedDevice(userInfo.id)) {













                        ComponentName who = getProfileOwnerAsUser(userInfo.id);













                        ComponentName who = getProfileOwner(userInfo.id);















                        return getActiveAdminUncheckedLocked(who, userInfo.id);















                    }















                }










@@ -9407,7 +9413,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













            return null;















        }















        enforceManageUsers();













        ComponentName profileOwner = getProfileOwnerAsUser(userHandle);













        ComponentName profileOwner = getProfileOwner(userHandle);















        if (profileOwner == null) {















            return null;















        }










@@ -9804,7 +9810,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













            return false;















        }



























        final ComponentName profileOwner = getProfileOwnerAsUser(userId);













        final ComponentName profileOwner = getProfileOwner(userId);















        if (profileOwner == null) {















            return false;















        }










@@ -13437,7 +13443,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













            // Managed-profiles cannot be setup on the system user.















            return CODE_SPLIT_SYSTEM_USER_DEVICE_SYSTEM_USER;















        }













        if (getProfileOwnerAsUser(callingUserId) != null) {













        if (getProfileOwner(callingUserId) != null) {















            // Managed user cannot have a managed profile.















            return CODE_USER_HAS_PROFILE_OWNER;















        }










@@ -14091,7 +14097,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













            return true;















        }



























        final ComponentName profileOwner = getProfileOwnerAsUser(userId);













        final ComponentName profileOwner = getProfileOwner(userId);















        if (profileOwner == null) {















            return false;















        }










@@ -14301,7 +14307,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {



























        enforceUserUnlocked(userId);



























        final ComponentName profileOwner = getProfileOwnerAsUser(userId);













        final ComponentName profileOwner = getProfileOwner(userId);















        if (profileOwner != null && packageName.equals(profileOwner.getPackageName())) {















            throw new IllegalArgumentException("Cannot uninstall a package with a profile owner");















        }










@@ -15839,7 +15845,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager {













            final List<ActiveAdmin> admins = new ArrayList<>();















            int[] users = mUserManager.getProfileIdsWithDisabled(UserHandle.getCallingUserId());















            for (int i = 0; i < users.length; i++) {













                final ComponentName componentName = getProfileOwnerAsUser(users[i]);













                final ComponentName componentName = getProfileOwner(users[i]);















                if (componentName != null) {















                    ActiveAdmin admin = getActiveAdminUncheckedLocked(componentName, users[i]);















                    if (admin != null) {

























services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerTest.java



+0
−10




























Original line number
Diff line number
Diff line








@@ -671,7 +671,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {













     */















    public void testRemoveActiveAdmin_fromDifferentUserWithINTERACT_ACROSS_USERS_FULL() {















        mContext.callerPermissions.add(android.Manifest.permission.MANAGE_DEVICE_ADMINS);













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);































        // Add admin1.


























@@ -2732,7 +2731,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {













        mContext.callerPermissions.add(permission.MANAGE_DEVICE_ADMINS);















        mContext.callerPermissions.add(permission.MANAGE_PROFILE_AND_DEVICE_OWNERS);















        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS_FULL);













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);































        // Check that the system user is unaffiliated.















        mContext.binder.callingUid = DpmMockContext.CALLER_SYSTEM_USER_UID;










@@ -4269,8 +4267,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {













    }































    public void testGetBindDeviceAdminTargetUsers() throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);





























        // Setup device owner.















        mContext.binder.callingUid = DpmMockContext.CALLER_SYSTEM_USER_UID;















        setupDeviceOwner();










@@ -6102,7 +6098,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {













    }































    public void testGetAllCrossProfilePackages_notSet_returnsEmpty() throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);















        addManagedProfile(admin1, mServiceContext.binder.callingUid, admin1);















        mContext.packageName = admin1.getPackageName();

























@@ -6114,7 +6109,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {





























    public void testGetAllCrossProfilePackages_notSet_dpmsReinitialized_returnsEmpty()















            throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);















        addManagedProfile(admin1, mServiceContext.binder.callingUid, admin1);















        mContext.packageName = admin1.getPackageName();

























@@ -6126,7 +6120,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {













    }































    public void testGetAllCrossProfilePackages_whenSet_returnsCombinedSet() throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);















        addManagedProfile(admin1, mServiceContext.binder.callingUid, admin1);















        final Set<String> packages = Sets.newSet("TEST_PACKAGE", "TEST_COMMON_PACKAGE");















        mContext.packageName = admin1.getPackageName();









@@ -6144,7 +6137,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {





























    public void testGetAllCrossProfilePackages_whenSet_dpmsReinitialized_returnsCombinedSet()















            throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);















        addManagedProfile(admin1, mServiceContext.binder.callingUid, admin1);















        final Set<String> packages = Sets.newSet("TEST_PACKAGE", "TEST_COMMON_PACKAGE");















        mContext.packageName = admin1.getPackageName();










@@ -6316,8 +6308,6 @@ public class DevicePolicyManagerTest extends DpmTestBase {





























    public void testSetAccountTypesWithManagementDisabledOnOrgOwnedManagedProfile()















            throws Exception {













        mContext.callerPermissions.add(permission.INTERACT_ACROSS_USERS);





























        final int managedProfileUserId = 15;















        final int managedProfileAdminUid = UserHandle.getUid(managedProfileUserId, 19436);