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

Commit f7b7e8bc authored by Jeff Sharkey's avatar Jeff Sharkey Committed by Android (Google) Code Review
Browse files

Merge changes from topic "oct11" 

* changes:
  BroadcastQueue: Handle successor ProcessRecords.
  More robust ProcessRecord routing for resultTo.
parents abeda2bf af59fa3b

core/java/android/app/ActivityManagerInternal.java

+5 −4
Original line number Diff line number Diff line
@@ -420,10 +420,11 @@ public abstract class ActivityManagerInternal { 


    public abstract int broadcastIntentInPackage(String packageName, @Nullable String featureId,

            int uid, int realCallingUid, int realCallingPid, Intent intent, String resolvedType,

            IIntentReceiver resultTo, int resultCode, String resultData, Bundle resultExtras,

            String requiredPermission, Bundle bOptions, boolean serialized, boolean sticky,

            @UserIdInt int userId, boolean allowBackgroundActivityStarts,

            @Nullable IBinder backgroundActivityStartsToken, @Nullable int[] broadcastAllowList);

            IApplicationThread resultToThread, IIntentReceiver resultTo, int resultCode,

            String resultData, Bundle resultExtras, String requiredPermission, Bundle bOptions,

            boolean serialized, boolean sticky, @UserIdInt int userId,

            boolean allowBackgroundActivityStarts, @Nullable IBinder backgroundActivityStartsToken,

            @Nullable int[] broadcastAllowList);



    public abstract ComponentName startServiceInPackage(int uid, Intent service,

            String resolvedType, boolean fgRequired, String callingPackage,

core/java/android/app/IActivityManager.aidl

+2 −1
Original line number Diff line number Diff line
@@ -556,7 +556,8 @@ interface IActivityManager { 
    void startConfirmDeviceCredentialIntent(in Intent intent, in Bundle options);

    @UnsupportedAppUsage(maxTargetSdk = 30, trackingBug = 170729553)

    void sendIdleJobTrigger();

    int sendIntentSender(in IIntentSender target, in IBinder whitelistToken, int code,

    int sendIntentSender(in IApplicationThread caller, in IIntentSender target,

            in IBinder whitelistToken, int code,

            in Intent intent, in String resolvedType, in IIntentReceiver finishedReceiver,

            in String requiredPermission, in Bundle options);

    boolean isBackgroundRestricted(in String packageName);

core/java/android/app/PendingIntent.java

+3 −1
Original line number Diff line number Diff line
@@ -1009,7 +1009,9 @@ public final class PendingIntent implements Parcelable { 
                options = activityOptions.toBundle();

            }



            return ActivityManager.getService().sendIntentSender(

            final IApplicationThread app = ActivityThread.currentActivityThread()

                    .getApplicationThread();

            return ActivityManager.getService().sendIntentSender(app,

                    mTarget, mWhitelistToken, code, intent, resolvedType,

                    onFinished != null

                            ? new FinishedDispatcher(this, onFinished, handler)

core/java/android/content/IntentSender.java

+5 −1
Original line number Diff line number Diff line
@@ -18,6 +18,8 @@ package android.content; 


import android.annotation.Nullable;

import android.app.ActivityManager;

import android.app.ActivityThread;

import android.app.IApplicationThread;

import android.app.ActivityManager.PendingIntentInfo;

import android.compat.annotation.UnsupportedAppUsage;

import android.os.Bundle;
@@ -194,7 +196,9 @@ public class IntentSender implements Parcelable { 
            String resolvedType = intent != null ?

                    intent.resolveTypeIfNeeded(context.getContentResolver())

                    : null;

            int res = ActivityManager.getService().sendIntentSender(mTarget, mWhitelistToken,

            final IApplicationThread app = ActivityThread.currentActivityThread()

                    .getApplicationThread();

            int res = ActivityManager.getService().sendIntentSender(app, mTarget, mWhitelistToken,

                    code, intent, resolvedType,

                    onFinished != null

                            ? new FinishedDispatcher(this, onFinished, handler)

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

+58 −38
Original line number Diff line number Diff line
@@ -3902,12 +3902,13 @@ public class ActivityManagerService extends IActivityManager.Stub 
                    if (isInstantApp) {

                        intent.putExtra(Intent.EXTRA_PACKAGE_NAME, packageName);

                        broadcastIntentInPackage("android", null, SYSTEM_UID, uid, pid, intent,

                                null, null, 0, null, null, permission.ACCESS_INSTANT_APPS, null,

                                false, false, resolvedUserId, false, null, visibilityAllowList);

                                null, null, null, 0, null, null, permission.ACCESS_INSTANT_APPS,

                                null, false, false, resolvedUserId, false, null,

                                visibilityAllowList);

                    } else {

                        broadcastIntentInPackage("android", null, SYSTEM_UID, uid, pid, intent,

                                null, null, 0, null, null, null, null, false, false, resolvedUserId,

                                false, null, visibilityAllowList);

                                null, null, null, 0, null, null, null, null, false, false,

                                resolvedUserId, false, null, visibilityAllowList);

                    }















                    if (observer != null) {










@@ -4444,7 +4445,8 @@ public class ActivityManagerService extends IActivityManager.Stub













        intent.putExtra(Intent.EXTRA_UID, uid);















        intent.putExtra(Intent.EXTRA_USER_HANDLE, userId);















        broadcastIntentLocked(null /* callerApp */, null /* callerPackage */,













                null /* callerFeatureId */, intent, null /* resolvedType */, null /* resultTo */,













                null /* callerFeatureId */, intent, null /* resolvedType */,













                null /* resultToApp */, null /* resultTo */,















                0 /* resultCode */, null /* resultData */, null /* resultExtras */,















                null /* requiredPermissions */, null /* excludedPermissions */,















                null /* excludedPackages */, OP_NONE, null /* bOptions */, false /* ordered */,










@@ -5515,12 +5517,12 @@ public class ActivityManagerService extends IActivityManager.Stub













    }





























    @Override













    public int sendIntentSender(IIntentSender target, IBinder allowlistToken, int code,













            Intent intent, String resolvedType,













    public int sendIntentSender(IApplicationThread caller, IIntentSender target,













            IBinder allowlistToken, int code, Intent intent, String resolvedType,















            IIntentReceiver finishedReceiver, String requiredPermission, Bundle options) {















        if (target instanceof PendingIntentRecord) {













            return ((PendingIntentRecord)target).sendWithResult(code, intent, resolvedType,













                    allowlistToken, finishedReceiver, requiredPermission, options);













            return ((PendingIntentRecord) target).sendWithResult(caller, code, intent,













                    resolvedType, allowlistToken, finishedReceiver, requiredPermission, options);















        } else {















            if (intent == null) {















                // Weird case: someone has given us their own custom IIntentSender, and now










@@ -13586,8 +13588,8 @@ public class ActivityManagerService extends IActivityManager.Stub













                    BroadcastQueue queue = broadcastQueueForIntent(intent);















                    BroadcastRecord r = new BroadcastRecord(queue, intent, null,















                            null, null, -1, -1, false, null, null, null, null, OP_NONE, null,













                            receivers, null, 0, null, null, false, true, true, -1, false, null,













                            false /* only PRE_BOOT_COMPLETED should be exempt, no stickies */,













                            receivers, null, null, 0, null, null, false, true, true, -1, false,













                            null, false /* only PRE_BOOT_COMPLETED should be exempt, no stickies */,















                            null /* filterExtrasForReceiver */);















                    queue.enqueueBroadcastLocked(r);















                }










@@ -13842,9 +13844,9 @@ public class ActivityManagerService extends IActivityManager.Stub













            boolean sticky, int callingPid,















            int callingUid, int realCallingUid, int realCallingPid, int userId) {















        return broadcastIntentLocked(callerApp, callerPackage, callerFeatureId, intent,













                resolvedType, resultTo, resultCode, resultData, resultExtras, requiredPermissions,













                excludedPermissions, excludedPackages, appOp, bOptions, ordered, sticky, callingPid,













                callingUid, realCallingUid, realCallingPid, userId,













                resolvedType, null, resultTo, resultCode, resultData, resultExtras,













                requiredPermissions, excludedPermissions, excludedPackages, appOp, bOptions,













                ordered, sticky, callingPid, callingUid, realCallingUid, realCallingPid, userId,















                false /* allowBackgroundActivityStarts */,















                null /* tokenNeededForBackgroundActivityStarts */,















                null /* broadcastAllowList */, null /* filterExtrasForReceiver */);









@@ -13853,7 +13855,7 @@ public class ActivityManagerService extends IActivityManager.Stub













    @GuardedBy("this")















    final int broadcastIntentLocked(ProcessRecord callerApp, String callerPackage,















            @Nullable String callerFeatureId, Intent intent, String resolvedType,













            IIntentReceiver resultTo, int resultCode, String resultData,













            ProcessRecord resultToApp, IIntentReceiver resultTo, int resultCode, String resultData,















            Bundle resultExtras, String[] requiredPermissions,















            String[] excludedPermissions, String[] excludedPackages, int appOp, Bundle bOptions,















            boolean ordered, boolean sticky, int callingPid, int callingUid,









@@ -13862,6 +13864,18 @@ public class ActivityManagerService extends IActivityManager.Stub













            @Nullable IBinder backgroundActivityStartsToken,















            @Nullable int[] broadcastAllowList,















            @Nullable BiFunction<Integer, Bundle, Bundle> filterExtrasForReceiver) {













        if ((resultTo != null) && (resultToApp == null)) {













            if (resultTo.asBinder() instanceof BinderProxy) {













                // Warn when requesting results without a way to deliver them













                Slog.wtf(TAG, "Sending broadcast " + intent.getAction()













                        + " with resultTo requires resultToApp", new Throwable());













            } else {













                // If not a BinderProxy above, then resultTo is an in-process













                // receiver, so splice in system_server process













                resultToApp = getProcessRecordLocked("system", SYSTEM_UID);













            }













        }



























        intent = new Intent(intent);





























        final boolean callerInstantApp = isInstantApp(callerApp, callerPackage, callingUid);










@@ -14461,8 +14475,8 @@ public class ActivityManagerService extends IActivityManager.Stub













            BroadcastRecord r = new BroadcastRecord(queue, intent, callerApp, callerPackage,















                    callerFeatureId, callingPid, callingUid, callerInstantApp, resolvedType,















                    requiredPermissions, excludedPermissions, excludedPackages, appOp, brOptions,













                    registeredReceivers, resultTo, resultCode, resultData, resultExtras, ordered,













                    sticky, false, userId, allowBackgroundActivityStarts,













                    registeredReceivers, resultToApp, resultTo, resultCode, resultData,













                    resultExtras, ordered, sticky, false, userId, allowBackgroundActivityStarts,















                    backgroundActivityStartsToken, timeoutExempt, filterExtrasForReceiver);















            if (DEBUG_BROADCAST) Slog.v(TAG_BROADCAST, "Enqueueing parallel broadcast " + r);















            queue.enqueueBroadcastLocked(r);










@@ -14555,7 +14569,7 @@ public class ActivityManagerService extends IActivityManager.Stub













            BroadcastRecord r = new BroadcastRecord(queue, intent, callerApp, callerPackage,















                    callerFeatureId, callingPid, callingUid, callerInstantApp, resolvedType,















                    requiredPermissions, excludedPermissions, excludedPackages, appOp, brOptions,













                    receivers, resultTo, resultCode, resultData, resultExtras,













                    receivers, resultToApp, resultTo, resultCode, resultData, resultExtras,















                    ordered, sticky, false, userId, allowBackgroundActivityStarts,















                    backgroundActivityStartsToken, timeoutExempt, filterExtrasForReceiver);
























@@ -14680,6 +14694,9 @@ public class ActivityManagerService extends IActivityManager.Stub













            final int callingPid = Binder.getCallingPid();















            final int callingUid = Binder.getCallingUid();



























            // We're delivering the result to the caller













            final ProcessRecord resultToApp = callerApp;



























            // Non-system callers can't declare that a broadcast is alarm-related.















            // The PendingIntent invocation case is handled in PendingIntentRecord.















            if (bOptions != null && callingUid != SYSTEM_UID) {









@@ -14697,9 +14714,10 @@ public class ActivityManagerService extends IActivityManager.Stub













            try {















                return broadcastIntentLocked(callerApp,















                        callerApp != null ? callerApp.info.packageName : null, callingFeatureId,













                        intent, resolvedType, resultTo, resultCode, resultData, resultExtras,













                        requiredPermissions, excludedPermissions, excludedPackages, appOp, bOptions,













                        serialized, sticky, callingPid, callingUid, callingUid, callingPid, userId);













                        intent, resolvedType, resultToApp, resultTo, resultCode, resultData,













                        resultExtras, requiredPermissions, excludedPermissions, excludedPackages,













                        appOp, bOptions, serialized, sticky, callingPid, callingUid, callingUid,













                        callingPid, userId, false, null, null, null);















            } finally {















                Binder.restoreCallingIdentity(origId);















            }









@@ -14709,11 +14727,10 @@ public class ActivityManagerService extends IActivityManager.Stub













    // Not the binder call surface















    int broadcastIntentInPackage(String packageName, @Nullable String featureId, int uid,















            int realCallingUid, int realCallingPid, Intent intent, String resolvedType,













            IIntentReceiver resultTo, int resultCode, String resultData, Bundle resultExtras,













            String requiredPermission, Bundle bOptions, boolean serialized, boolean sticky,













            int userId, boolean allowBackgroundActivityStarts,













            @Nullable IBinder backgroundActivityStartsToken,













            @Nullable int[] broadcastAllowList) {













            ProcessRecord resultToApp, IIntentReceiver resultTo, int resultCode,













            String resultData, Bundle resultExtras, String requiredPermission, Bundle bOptions,













            boolean serialized, boolean sticky, int userId, boolean allowBackgroundActivityStarts,













            @Nullable IBinder backgroundActivityStartsToken, @Nullable int[] broadcastAllowList) {















        synchronized(this) {















            intent = verifyBroadcastLocked(intent);























@@ -14722,9 +14739,9 @@ public class ActivityManagerService extends IActivityManager.Stub













                    : new String[] {requiredPermission};















            try {















                return broadcastIntentLocked(null, packageName, featureId, intent, resolvedType,













                        resultTo, resultCode, resultData, resultExtras, requiredPermissions, null,













                        null, OP_NONE, bOptions, serialized, sticky, -1, uid, realCallingUid,













                        realCallingPid, userId, allowBackgroundActivityStarts,













                        resultToApp, resultTo, resultCode, resultData, resultExtras,













                        requiredPermissions, null, null, OP_NONE, bOptions, serialized, sticky, -1,













                        uid, realCallingUid, realCallingPid, userId, allowBackgroundActivityStarts,















                        backgroundActivityStartsToken, broadcastAllowList,















                        null /* filterExtrasForReceiver */);















            } finally {










@@ -17253,16 +17270,18 @@ public class ActivityManagerService extends IActivityManager.Stub













        @Override















        public int broadcastIntentInPackage(String packageName, @Nullable String featureId, int uid,















                int realCallingUid, int realCallingPid, Intent intent, String resolvedType,













                IIntentReceiver resultTo, int resultCode, String resultData, Bundle resultExtras,













                String requiredPermission, Bundle bOptions, boolean serialized, boolean sticky,













                int userId, boolean allowBackgroundActivityStarts,













                IApplicationThread resultToThread, IIntentReceiver resultTo, int resultCode,













                String resultData, Bundle resultExtras, String requiredPermission, Bundle bOptions,













                boolean serialized, boolean sticky, int userId,













                boolean allowBackgroundActivityStarts,















                @Nullable IBinder backgroundActivityStartsToken,















                @Nullable int[] broadcastAllowList) {















            synchronized (ActivityManagerService.this) {













                final ProcessRecord resultToApp = getRecordForAppLOSP(resultToThread);















                return ActivityManagerService.this.broadcastIntentInPackage(packageName, featureId,













                        uid, realCallingUid, realCallingPid, intent, resolvedType, resultTo,













                        resultCode, resultData, resultExtras, requiredPermission, bOptions,













                        serialized, sticky, userId, allowBackgroundActivityStarts,













                        uid, realCallingUid, realCallingPid, intent, resolvedType, resultToApp,













                        resultTo, resultCode, resultData, resultExtras, requiredPermission,













                        bOptions, serialized, sticky, userId, allowBackgroundActivityStarts,















                        backgroundActivityStartsToken, broadcastAllowList);















            }















        }









@@ -17283,7 +17302,8 @@ public class ActivityManagerService extends IActivityManager.Stub













                try {















                    return ActivityManagerService.this.broadcastIntentLocked(null /*callerApp*/,















                            null /*callerPackage*/, null /*callingFeatureId*/, intent,













                            null /*resolvedType*/, resultTo, 0 /*resultCode*/, null /*resultData*/,













                            null /* resolvedType */, null /* resultToApp */, resultTo,













                            0 /* resultCode */, null /* resultData */,















                            null /* resultExtras */, requiredPermissions,















                            null /*excludedPermissions*/, null /*excludedPackages*/,















                            AppOpsManager.OP_NONE, bOptions /*options*/, serialized,










@@ -17897,7 +17917,7 @@ public class ActivityManagerService extends IActivityManager.Stub













        public int sendIntentSender(IIntentSender target, IBinder allowlistToken, int code,















                Intent intent, String resolvedType,















                IIntentReceiver finishedReceiver, String requiredPermission, Bundle options) {













            return ActivityManagerService.this.sendIntentSender(target, allowlistToken, code,













            return ActivityManagerService.this.sendIntentSender(null, target, allowlistToken, code,















                    intent, resolvedType, finishedReceiver, requiredPermission, options);















        }