Merge "[RESTRICT AUTOMERGE] Trim the activity info of another uid if no privilege" into sc-dev (8c8286d8) · Commits · e / os / android_frameworks_base

services/core/java/com/android/server/wm/AppTaskImpl.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -84,7 +84,7 @@ class AppTaskImpl extends IAppTask.Stub {
		throw new IllegalArgumentException("Unable to find task ID " + mTaskId);
		}
		return mService.getRecentTasks().createRecentTaskInfo(task,
		false /* stripExtras */);
		false /* stripExtras /, true / getTasksAllowed */);
		} finally {
		Binder.restoreCallingIdentity(origId);
		}

services/core/java/com/android/server/wm/RecentTasks.java

+6 −2

Original line number	Diff line number	Diff line
		@@ -974,7 +974,7 @@ class RecentTasks {
		continue;
		}

		res.add(createRecentTaskInfo(task, true /* stripExtras */));
		res.add(createRecentTaskInfo(task, true /* stripExtras */, getTasksAllowed));
		}
		return res;
		}
		@@ -1886,7 +1886,8 @@ class RecentTasks {
		/**
		* Creates a new RecentTaskInfo from a Task.
		*/
		ActivityManager.RecentTaskInfo createRecentTaskInfo(Task tr, boolean stripExtras) {
		ActivityManager.RecentTaskInfo createRecentTaskInfo(Task tr, boolean stripExtras,
		boolean getTasksAllowed) {
		final ActivityManager.RecentTaskInfo rti = new ActivityManager.RecentTaskInfo();
		// If the recent Task is detached, we consider it will be re-attached to the default
		// TaskDisplayArea because we currently only support recent overview in the default TDA.
		@@ -1898,6 +1899,9 @@ class RecentTasks {
		rti.id = rti.isRunning ? rti.taskId : INVALID_TASK_ID;
		rti.persistentId = rti.taskId;
		rti.lastSnapshotData.set(tr.mLastTaskSnapshotData);
		if (!getTasksAllowed) {
		Task.trimIneffectiveInfo(tr, rti);
		}

		// Fill in organized child task info for the task created by organizer.
		if (tr.mCreatedByOrganizer) {

services/core/java/com/android/server/wm/RunningTasks.java

+4 −0

Original line number	Diff line number	Diff line
		@@ -137,6 +137,10 @@ class RunningTasks {
		task.fillTaskInfo(rti, !mKeepIntentExtra);
		// Fill in some deprecated values
		rti.id = rti.taskId;

		if (!mAllowed) {
		Task.trimIneffectiveInfo(task, rti);
		}
		return rti;
		}
		}

services/core/java/com/android/server/wm/Task.java

+21 −0

Original line number	Diff line number	Diff line
		@@ -4156,6 +4156,27 @@ class Task extends WindowContainer<WindowContainer> {
		info.mTopActivityLocusId = topRecord != null ? topRecord.getLocusId() : null;
		}

		/**
		* Removes the activity info if the activity belongs to a different uid, which is
		* different from the app that hosts the task.
		*/
		static void trimIneffectiveInfo(Task task, TaskInfo info) {
		final ActivityRecord baseActivity = task.getActivity(r -> !r.finishing,
		false /* traverseTopToBottom */);
		final int baseActivityUid =
		baseActivity != null ? baseActivity.getUid() : task.effectiveUid;

		if (info.topActivityInfo != null
		&& task.effectiveUid != info.topActivityInfo.applicationInfo.uid) {
		info.topActivity = null;
		info.topActivityInfo = null;
		}

		if (task.effectiveUid != baseActivityUid) {
		info.baseActivity = null;
		}
		}

		@Nullable PictureInPictureParams getPictureInPictureParams() {
		return getPictureInPictureParams(getTopMostTask());
		}

services/tests/wmtests/src/com/android/server/wm/RecentTasksTest.java

+22 −5

Original line number	Diff line number	Diff line
		@@ -30,6 +30,7 @@ import static android.content.Intent.FLAG_ACTIVITY_NEW_TASK;
		import static android.content.pm.ActivityInfo.LAUNCH_MULTIPLE;
		import static android.content.pm.ActivityInfo.LAUNCH_SINGLE_INSTANCE;
		import static android.content.res.Configuration.ORIENTATION_PORTRAIT;
		import static android.os.Process.NOBODY_UID;

		import static com.android.dx.mockito.inline.extended.ExtendedMockito.doNothing;
		import static com.android.dx.mockito.inline.extended.ExtendedMockito.doReturn;
		@@ -1145,21 +1146,35 @@ public class RecentTasksTest extends WindowTestsBase {

		@Test
		public void testCreateRecentTaskInfo_detachedTask() {
		final Task task = createTaskBuilder(".Task").setCreateActivity(true).build();
		final Task task = createTaskBuilder(".Task").build();
		new ActivityBuilder(mSupervisor.mService)
		.setTask(task)
		.setUid(NOBODY_UID)
		.setComponent(new ComponentName("com.foo", ".BarActivity"))
		.build();
		final TaskDisplayArea tda = task.getDisplayArea();

		assertTrue(task.isAttached());
		assertTrue(task.supportsMultiWindow());

		RecentTaskInfo info = mRecentTasks.createRecentTaskInfo(task, true);
		RecentTaskInfo info = mRecentTasks.createRecentTaskInfo(task, true /* stripExtras */,
		true /* getTasksAllowed */);

		assertTrue(info.supportsMultiWindow);
		assertTrue(info.supportsSplitScreenMultiWindow);

		info = mRecentTasks.createRecentTaskInfo(task, true /* stripExtras */,
		false /* getTasksAllowed */);

		assertTrue(info.topActivity == null);
		assertTrue(info.topActivityInfo == null);
		assertTrue(info.baseActivity == null);

		// The task can be put in split screen even if it is not attached now.
		task.removeImmediately();

		info = mRecentTasks.createRecentTaskInfo(task, true);
		info = mRecentTasks.createRecentTaskInfo(task, true /* stripExtras */,
		true /* getTasksAllowed */);

		assertTrue(info.supportsMultiWindow);
		assertTrue(info.supportsSplitScreenMultiWindow);
		@@ -1169,7 +1184,8 @@ public class RecentTasksTest extends WindowTestsBase {
		doReturn(false).when(tda).supportsNonResizableMultiWindow();
		doReturn(false).when(task).isResizeable();

		info = mRecentTasks.createRecentTaskInfo(task, true);
		info = mRecentTasks.createRecentTaskInfo(task, true /* stripExtras */,
		true /* getTasksAllowed */);

		assertFalse(info.supportsMultiWindow);
		assertFalse(info.supportsSplitScreenMultiWindow);
		@@ -1178,7 +1194,8 @@ public class RecentTasksTest extends WindowTestsBase {
		// the device supports it.
		doReturn(true).when(tda).supportsNonResizableMultiWindow();

		info = mRecentTasks.createRecentTaskInfo(task, true);
		info = mRecentTasks.createRecentTaskInfo(task, true /* stripExtras */,
		true /* getTasksAllowed */);

		assertTrue(info.supportsMultiWindow);
		assertTrue(info.supportsSplitScreenMultiWindow);