From d67b187dcc5bde21ef9c137529ade4d375f13d4c Mon Sep 17 00:00:00 2001 From: Makoto Onuki Date: Thu, 21 Jul 2016 16:26:36 -0700 Subject: [PATCH] Fix realCallingUid/Pid for startActivities() They now match startActivityMayWait(). Bug 30277795 Change-Id: I883d74bf02cb79a19e2139134aa9be0296787fed --- .../core/java/com/android/server/am/ActivityStarter.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/am/ActivityStarter.java b/services/core/java/com/android/server/am/ActivityStarter.java index 53289be4294a5..5252446990825 100644 --- a/services/core/java/com/android/server/am/ActivityStarter.java +++ b/services/core/java/com/android/server/am/ActivityStarter.java @@ -895,13 +895,15 @@ class ActivityStarter { throw new IllegalArgumentException("intents are length different than resolvedTypes"); } + final int realCallingPid = Binder.getCallingPid(); + final int realCallingUid = Binder.getCallingUid(); int callingPid; if (callingUid >= 0) { callingPid = -1; } else if (caller == null) { - callingPid = Binder.getCallingPid(); - callingUid = Binder.getCallingUid(); + callingPid = realCallingPid; + callingUid = realCallingUid; } else { callingPid = callingUid = -1; } @@ -942,7 +944,8 @@ class ActivityStarter { i == intents.length - 1 ? bOptions : null); int res = startActivityLocked(caller, intent, null /*ephemeralIntent*/, resolvedTypes[i], aInfo, null /*rInfo*/, null, null, resultTo, null, -1, - callingPid, callingUid, callingPackage, callingPid, callingUid, 0, + callingPid, callingUid, callingPackage, + realCallingPid, realCallingUid, 0, options, false, componentSpecified, outActivity, null, null); if (res < 0) { return res;