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

Commit 10286bb0 authored by Martijn Coenen's avatar Martijn Coenen Committed by Android (Google) Code Review
Browse files

Merge "Fix IsolatedUidAllocator for non-primary users."

parents fa72dfb4 c9a0df2d

services/core/java/com/android/server/am/ProcessList.java

+1 −3
Original line number Diff line number Diff line
@@ -461,9 +461,7 @@ public final class ProcessList { 


        @GuardedBy("ProcessList.this.mService")

        void freeIsolatedUidLocked(int uid) {

            // Strip out userId

            final int appId = UserHandle.getAppId(uid);

            mUidUsed.delete(appId);

            mUidUsed.delete(uid);

        }

    };

services/tests/servicestests/src/com/android/server/am/ActivityManagerServiceTest.java

+12 −1
Original line number Diff line number Diff line
@@ -349,10 +349,21 @@ public class ActivityManagerServiceTest { 
        verifyUidRangesNoOverlap(range, range2);

        verifyIsolatedUidAllocator(range2);



        // Free both, then try to allocate the maximum number of UID ranges

        // Free both

        allocator.freeUidRangeLocked(appInfo);

        allocator.freeUidRangeLocked(appInfo2);



        // Verify for a secondary user

        ApplicationInfo appInfo3 = new ApplicationInfo();

        appInfo3.processName = "com.android.test.app";

        appInfo3.uid = 1010000;

        final IsolatedUidRange range3 = allocator.getOrCreateIsolatedUidRangeLocked(

                appInfo3.processName, appInfo3.uid);

        validateAppZygoteIsolatedUidRange(range3);

        verifyIsolatedUidAllocator(range3);



        allocator.freeUidRangeLocked(appInfo3);

        // Try to allocate the maximum number of UID ranges

        int maxNumUidRanges = (Process.LAST_APP_ZYGOTE_ISOLATED_UID

                - Process.FIRST_APP_ZYGOTE_ISOLATED_UID + 1) / Process.NUM_UIDS_PER_APP_ZYGOTE;

        for (int i = 0; i < maxNumUidRanges; i++) {