Merge "Early exit when target app ID < FIRST_APP_UID" into rvc-dev am: d65f6e8fb4

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11885735

Change-Id: I69a5178ac879e967516cdd3ad966907f2970abcd
This commit is contained in:
TreeHugger Robot
2020-06-18 05:43:59 +00:00
committed by Automerger Merge Worker
2 changed files with 3 additions and 1 deletions

View File

@@ -861,6 +861,7 @@ public class AppsFilter {
try { try {
int callingAppId = UserHandle.getAppId(callingUid); int callingAppId = UserHandle.getAppId(callingUid);
if (callingAppId < Process.FIRST_APPLICATION_UID if (callingAppId < Process.FIRST_APPLICATION_UID
|| targetPkgSetting.appId < Process.FIRST_APPLICATION_UID
|| callingAppId == targetPkgSetting.appId) { || callingAppId == targetPkgSetting.appId) {
return false; return false;
} }

View File

@@ -806,7 +806,8 @@ public class AppsFilterTest {
final SparseArray<int[]> systemFilter = final SparseArray<int[]> systemFilter =
appsFilter.getVisibilityWhitelist(system, USER_ARRAY, mExisting); appsFilter.getVisibilityWhitelist(system, USER_ARRAY, mExisting);
assertThat(toList(systemFilter.get(SYSTEM_USER)), empty()); assertThat(toList(systemFilter.get(SYSTEM_USER)),
contains(seesNothingAppId, hasProviderAppId, queriesProviderAppId));
final SparseArray<int[]> seesNothingFilter = final SparseArray<int[]> seesNothingFilter =
appsFilter.getVisibilityWhitelist(seesNothing, USER_ARRAY, mExisting); appsFilter.getVisibilityWhitelist(seesNothing, USER_ARRAY, mExisting);