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

Commit 2900cfda authored by Daniel Norman's avatar Daniel Norman Committed by Automerger Merge Worker
Browse files

Merge "Use PM#isSameApp instead of getPackageUidAsUser." into udc-dev am: 53fd9812 

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



Change-Id: I18d184f0aca0e08a0f76e713b2c9cfd188076093
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 59ec6341 53fd9812

services/accessibility/java/com/android/server/accessibility/AccessibilityManagerService.java

+1 −1
Original line number Diff line number Diff line
@@ -490,7 +490,7 @@ public class AccessibilityManagerService extends IAccessibilityManager.Stub 
                mMainHandler, context,

                new PolicyWarningUIController.NotificationController(context));

        mSecurityPolicy = new AccessibilitySecurityPolicy(policyWarningUIController, mContext,

                this);

                this, LocalServices.getService(PackageManagerInternal.class));

        mA11yWindowManager = new AccessibilityWindowManager(mLock, mMainHandler,

                mWindowManagerService, this, mSecurityPolicy, this, mTraceManager);

        mA11yDisplayListener = new AccessibilityDisplayListener(mContext, mMainHandler);

services/accessibility/java/com/android/server/accessibility/AccessibilitySecurityPolicy.java

+6 −7
Original line number Diff line number Diff line
@@ -29,6 +29,7 @@ import android.appwidget.AppWidgetManagerInternal; 
import android.content.ComponentName;

import android.content.Context;

import android.content.pm.PackageManager;

import android.content.pm.PackageManagerInternal;

import android.content.pm.ResolveInfo;

import android.content.pm.ServiceInfo;

import android.content.pm.UserInfo;
@@ -99,6 +100,7 @@ public class AccessibilitySecurityPolicy { 


    private final Context mContext;

    private final PackageManager mPackageManager;

    private final PackageManagerInternal mPackageManagerInternal;

    private final UserManager mUserManager;

    private final AppOpsManager mAppOpsManager;

    private final AccessibilityUserManager mAccessibilityUserManager;
@@ -116,10 +118,12 @@ public class AccessibilitySecurityPolicy { 
     */

    public AccessibilitySecurityPolicy(PolicyWarningUIController policyWarningUIController,

            @NonNull Context context,

            @NonNull AccessibilityUserManager a11yUserManager) {

            @NonNull AccessibilityUserManager a11yUserManager,

            @NonNull PackageManagerInternal packageManagerInternal) {

        mContext = context;

        mAccessibilityUserManager = a11yUserManager;

        mPackageManager = mContext.getPackageManager();

        mPackageManagerInternal = packageManagerInternal;

        mUserManager = (UserManager) mContext.getSystemService(Context.USER_SERVICE);

        mAppOpsManager = (AppOpsManager) context.getSystemService(Context.APP_OPS_SERVICE);

        mPolicyWarningUIController = policyWarningUIController;
@@ -513,12 +517,7 @@ public class AccessibilitySecurityPolicy { 
    private boolean isValidPackageForUid(String packageName, int uid) {

        final long token = Binder.clearCallingIdentity();

        try {

            // Since we treat calls from a profile as if made by its parent, using

            // MATCH_ANY_USER to query the uid of the given package name.

            return uid == mPackageManager.getPackageUidAsUser(

                    packageName, PackageManager.MATCH_ANY_USER, UserHandle.getUserId(uid));

        } catch (PackageManager.NameNotFoundException e) {

            return false;

            return mPackageManagerInternal.isSameApp(packageName, uid, UserHandle.getUserId(uid));

        } finally {

            Binder.restoreCallingIdentity(token);

        }

services/tests/servicestests/src/com/android/server/accessibility/AccessibilitySecurityPolicyTest.java

+13 −12
Original line number Diff line number Diff line
@@ -146,7 +146,7 @@ public class AccessibilitySecurityPolicyTest { 
    @Mock

    private PolicyWarningUIController mPolicyWarningUIController;

    @Mock

    private PackageManagerInternal mPackageManagerInternal;

    private PackageManagerInternal mMockPackageManagerInternal;



    @Before

    public void setUp() {
@@ -158,7 +158,8 @@ public class AccessibilitySecurityPolicyTest { 
                R.dimen.accessibility_focus_highlight_stroke_width, 1);



        mA11ySecurityPolicy = new AccessibilitySecurityPolicy(

                mPolicyWarningUIController, mContext, mMockA11yUserManager);

                mPolicyWarningUIController, mContext, mMockA11yUserManager,

                mMockPackageManagerInternal);

        mA11ySecurityPolicy.setSendingNonA11yToolNotificationLocked(true);

        mA11ySecurityPolicy.setAccessibilityWindowManager(mMockA11yWindowManager);

        mA11ySecurityPolicy.setAppWidgetManager(mMockAppWidgetManager);
@@ -237,8 +238,8 @@ public class AccessibilitySecurityPolicyTest { 
    @Test

    public void resolveValidReportedPackage_uidAndPkgNameMatched_returnPkgName()

            throws PackageManager.NameNotFoundException {

        when(mMockPackageManager.getPackageUidAsUser(PACKAGE_NAME,

                PackageManager.MATCH_ANY_USER, TEST_USER_ID)).thenReturn(APP_UID);

        when(mMockPackageManagerInternal.isSameApp(PACKAGE_NAME, APP_UID, TEST_USER_ID))

                .thenReturn(true);



        assertEquals(mA11ySecurityPolicy.resolveValidReportedPackageLocked(

                PACKAGE_NAME, APP_UID, TEST_USER_ID, APP_PID),
@@ -257,8 +258,8 @@ public class AccessibilitySecurityPolicyTest { 


        when(mMockAppWidgetManager.getHostedWidgetPackages(widgetHostUid))

                .thenReturn(widgetPackages);

        when(mMockPackageManager.getPackageUidAsUser(hostPackageName, TEST_USER_ID))

                .thenReturn(widgetHostUid);

        when(mMockPackageManagerInternal.isSameApp(hostPackageName, widgetHostUid, TEST_USER_ID))

                .thenReturn(true);



        assertEquals(mA11ySecurityPolicy.resolveValidReportedPackageLocked(

                widgetPackageName, widgetHostUid, TEST_USER_ID, widgetHostPid),
@@ -272,8 +273,8 @@ public class AccessibilitySecurityPolicyTest { 
        final String[] uidPackages = {PACKAGE_NAME, PACKAGE_NAME2};

        when(mMockPackageManager.getPackagesForUid(APP_UID))

                .thenReturn(uidPackages);

        when(mMockPackageManager.getPackageUidAsUser(invalidPackageName, TEST_USER_ID))

                .thenThrow(PackageManager.NameNotFoundException.class);

        when(mMockPackageManagerInternal.isSameApp(invalidPackageName, APP_UID, TEST_USER_ID))

                .thenReturn(false);

        when(mMockAppWidgetManager.getHostedWidgetPackages(APP_UID))

                .thenReturn(new ArraySet<>());

        mContext.getTestablePermissions().setPermission(
@@ -292,8 +293,8 @@ public class AccessibilitySecurityPolicyTest { 
        final String[] uidPackages = {PACKAGE_NAME};

        when(mMockPackageManager.getPackagesForUid(APP_UID))

                .thenReturn(uidPackages);

        when(mMockPackageManager.getPackageUidAsUser(wantedPackageName, TEST_USER_ID))

                .thenReturn(wantedUid);

        when(mMockPackageManagerInternal.isSameApp(wantedPackageName, wantedUid, TEST_USER_ID))

                .thenReturn(true);

        when(mMockAppWidgetManager.getHostedWidgetPackages(APP_UID))

                .thenReturn(new ArraySet<>());

        mContext.getTestablePermissions().setPermission(
@@ -312,8 +313,8 @@ public class AccessibilitySecurityPolicyTest { 
        final String[] uidPackages = {PACKAGE_NAME};

        when(mMockPackageManager.getPackagesForUid(APP_UID))

                .thenReturn(uidPackages);

        when(mMockPackageManager.getPackageUidAsUser(wantedPackageName, TEST_USER_ID))

                .thenReturn(wantedUid);

        when(mMockPackageManagerInternal.isSameApp(wantedPackageName, wantedUid, TEST_USER_ID))

                .thenReturn(true);

        when(mMockAppWidgetManager.getHostedWidgetPackages(APP_UID))

                .thenReturn(new ArraySet<>());

        mContext.getTestablePermissions().setPermission(