From a935db15c22eba1d0f2409416774c3e5e4002367 Mon Sep 17 00:00:00 2001 From: Sergey Nikolaienkov Date: Tue, 22 Aug 2023 17:26:28 +0200 Subject: More comments in ContentProviderHelper Ever so slightly expand comments in the AM's ContentProviderHelper, which potentially could save some time other people trying to figure out why they are getting "Provider not found" errors. Bug: none Test: m Change-Id: Idd914dfbbcf83b10693d13fd2a4e6bf5038551e0 --- services/core/java/com/android/server/am/ContentProviderHelper.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/am/ContentProviderHelper.java b/services/core/java/com/android/server/am/ContentProviderHelper.java index 4cc147fcea03..9219623a031d 100644 --- a/services/core/java/com/android/server/am/ContentProviderHelper.java +++ b/services/core/java/com/android/server/am/ContentProviderHelper.java @@ -293,7 +293,8 @@ public class ContentProviderHelper { return holder; } - // Don't expose providers between normal apps and instant apps + // Don't expose providers between normal apps and instant apps; enforce limited + // package visibility (introduced in Android 11); etc. try { if (AppGlobals.getPackageManager() .resolveContentProvider(name, /*flags=*/ 0, userId) == null) { -- cgit v1.2.3-59-g8ed1b