From bad3a9b80bb90c9cb58ec36433db51f08dd40b1d Mon Sep 17 00:00:00 2001 From: Rhed Jao Date: Thu, 14 Apr 2022 17:30:36 +0800 Subject: [PATCH] Fix cross user package visibility leakage for PackageManager (2/n) API: - PackageManager#isPackageSuspended Bug: 214394701 Test: atest CrossUserPackageVisibilityTests Change-Id: Ibe8f7ed8b1dcaa47a6c35593a013957fa3293ab4 --- services/core/java/com/android/server/pm/ComputerEngine.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/pm/ComputerEngine.java b/services/core/java/com/android/server/pm/ComputerEngine.java index a29b1f8f39ca4..acb27ab5f72de 100644 --- a/services/core/java/com/android/server/pm/ComputerEngine.java +++ b/services/core/java/com/android/server/pm/ComputerEngine.java @@ -4990,7 +4990,7 @@ public class ComputerEngine implements Computer { enforceCrossUserPermission(callingUid, userId, true /* requireFullPermission */, false /* checkShell */, "isPackageSuspendedForUser for user " + userId); final PackageStateInternal ps = mSettings.getPackage(packageName); - if (ps == null || shouldFilterApplication(ps, callingUid, userId)) { + if (ps == null || shouldFilterApplicationIncludingUninstalled(ps, callingUid, userId)) { throw new IllegalArgumentException("Unknown target package: " + packageName); } return ps.getUserStateOrDefault(userId).isSuspended();