From a78a75684cfd771c1aee449208dd58a247c716fd Mon Sep 17 00:00:00 2001 From: Alex Buynytskyy Date: Tue, 30 Aug 2022 14:18:16 -0700 Subject: [PATCH] Fix for: NPE when intent.component set to ResolverActivity. Bug: 244294361 Fixes: 244294361 Fixes: 260291956 Test: atest PackageManagerTest Change-Id: I14fa8d8a9bb5020ebdf1f4b761b15e1d2f938920 --- .../server/pm/PackageManagerServiceUtils.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/pm/PackageManagerServiceUtils.java b/services/core/java/com/android/server/pm/PackageManagerServiceUtils.java index 15ba760dbca3a..714be185c9729 100644 --- a/services/core/java/com/android/server/pm/PackageManagerServiceUtils.java +++ b/services/core/java/com/android/server/pm/PackageManagerServiceUtils.java @@ -41,6 +41,7 @@ import android.compat.annotation.ChangeId; import android.compat.annotation.EnabledSince; import android.content.Context; import android.content.Intent; +import android.content.IntentFilter; import android.content.pm.ActivityInfo; import android.content.pm.ApplicationInfo; import android.content.pm.ComponentInfo; @@ -1133,9 +1134,14 @@ public class PackageManagerServiceUtils { continue; } - final boolean match = comp.getIntents().stream().anyMatch( - f -> IntentResolver.intentMatchesFilter(f.getIntentFilter(), intent, - resolvedType)); + boolean match = false; + for (int j = 0, size = comp.getIntents().size(); j < size; ++j) { + IntentFilter intentFilter = comp.getIntents().get(j).getIntentFilter(); + if (IntentResolver.intentMatchesFilter(intentFilter, intent, resolvedType)) { + match = true; + break; + } + } if (!match) { Slog.w(TAG, "Intent does not match component's intent filter: " + intent); Slog.w(TAG, "Access blocked: " + comp.getComponentName());