From 1282bc2f8d0eb389a6b6311c5b80c34f206498de Mon Sep 17 00:00:00 2001 From: Jorim Jaggi Date: Fri, 9 Mar 2018 18:14:48 +0100 Subject: [PATCH] Fix SafeActivityOptions construction in startActivityFromRecents Otherwise we'll check the wrong permissions and try to boost the wrong process. Test: Open app from recents, make sure Launcher gets scheduled as needed. Bug: 73555925 Change-Id: I2d1a7d0c881c0182127a5b25dce0ab1a308d5e63 --- .../java/com/android/server/am/ActivityManagerService.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/am/ActivityManagerService.java b/services/core/java/com/android/server/am/ActivityManagerService.java index c1c68e8554e85..96633da165089 100644 --- a/services/core/java/com/android/server/am/ActivityManagerService.java +++ b/services/core/java/com/android/server/am/ActivityManagerService.java @@ -5401,11 +5401,12 @@ public class ActivityManagerService extends IActivityManager.Stub final int callingPid = Binder.getCallingPid(); final int callingUid = Binder.getCallingUid(); + final SafeActivityOptions safeOptions = SafeActivityOptions.fromBundle(bOptions); final long origId = Binder.clearCallingIdentity(); try { synchronized (this) { return mStackSupervisor.startActivityFromRecents(callingPid, callingUid, taskId, - SafeActivityOptions.fromBundle(bOptions)); + safeOptions); } } finally { Binder.restoreCallingIdentity(origId);