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

Commit 947d63d1 authored by Patrick Baumann's avatar Patrick Baumann
Browse files

Early exit when target app ID < FIRST_APP_UID 

This change ensures that we also grant visibility to the framework and
other settings that are outside of the APPLICATION_UID space.

Fixes: 159152185
Test: atest com.android.server.pm.UserManagerTest#testAddManagedProfile_withDisallowedPackages
Change-Id: I00c8390dfa748f90c538a57243b472610886be7b
parent 7ecbe5b5

services/core/java/com/android/server/pm/AppsFilter.java

+1 −0
Original line number Diff line number Diff line
@@ -845,6 +845,7 @@ public class AppsFilter { 
        try {

            int callingAppId = UserHandle.getAppId(callingUid);

            if (callingAppId < Process.FIRST_APPLICATION_UID

                    || targetPkgSetting.appId < Process.FIRST_APPLICATION_UID

                    || callingAppId == targetPkgSetting.appId) {

                return false;

            }

services/tests/servicestests/src/com/android/server/pm/AppsFilterTest.java

+2 −1
Original line number Diff line number Diff line
@@ -806,7 +806,8 @@ public class AppsFilterTest { 


        final SparseArray<int[]> systemFilter =

                appsFilter.getVisibilityWhitelist(system, USER_ARRAY, mExisting);

        assertThat(toList(systemFilter.get(SYSTEM_USER)), empty());

        assertThat(toList(systemFilter.get(SYSTEM_USER)),

                contains(seesNothingAppId, hasProviderAppId, queriesProviderAppId));



        final SparseArray<int[]> seesNothingFilter =

                appsFilter.getVisibilityWhitelist(seesNothing, USER_ARRAY, mExisting);