mirror of
https://github.com/Divested-Mobile/DivestOS-Build.git
synced 2024-12-14 02:14:34 -05:00
a458d91293
no effective change Signed-off-by: Tavi <tavi@divested.dev>
112 lines
5.2 KiB
Diff
112 lines
5.2 KiB
Diff
From 227df78aae8974bcdde516d111189f082824a765 Mon Sep 17 00:00:00 2001
|
|
From: Chris Antol <cantol@google.com>
|
|
Date: Tue, 4 Jun 2024 17:00:46 +0000
|
|
Subject: [PATCH] Ignore fragment attr from ext authenticator resource
|
|
|
|
Bug: 341886134
|
|
Test: Unit Test
|
|
Test: Manual - see ticket for steps
|
|
Flag: EXEMPT <security>
|
|
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:2cb9b10ed97b1b9b29661115789605a762f3c2ef)
|
|
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:7e0b376b11318e1e79b31bac6aafc0c923868bc4)
|
|
Merged-In: Id91c2b3b6d16ba3702ee2cd6723365a4db52863b
|
|
Change-Id: Id91c2b3b6d16ba3702ee2cd6723365a4db52863b
|
|
---
|
|
.../accounts/AccountTypePreferenceLoader.java | 55 +++++++++++++++++++
|
|
1 file changed, 55 insertions(+)
|
|
|
|
diff --git a/src/com/android/settings/accounts/AccountTypePreferenceLoader.java b/src/com/android/settings/accounts/AccountTypePreferenceLoader.java
|
|
index c639d1df2eb..84386a38f08 100644
|
|
--- a/src/com/android/settings/accounts/AccountTypePreferenceLoader.java
|
|
+++ b/src/com/android/settings/accounts/AccountTypePreferenceLoader.java
|
|
@@ -32,6 +32,10 @@
|
|
import android.text.TextUtils;
|
|
import android.util.Log;
|
|
|
|
+import androidx.annotation.NonNull;
|
|
+import androidx.annotation.Nullable;
|
|
+import androidx.annotation.VisibleForTesting;
|
|
+import androidx.collection.ArraySet;
|
|
import androidx.preference.Preference;
|
|
import androidx.preference.Preference.OnPreferenceClickListener;
|
|
import androidx.preference.PreferenceFragmentCompat;
|
|
@@ -45,6 +49,8 @@
|
|
import com.android.settingslib.accounts.AuthenticatorHelper;
|
|
import com.android.settingslib.core.instrumentation.Instrumentable;
|
|
|
|
+import java.util.Set;
|
|
+
|
|
/**
|
|
* Class to load the preference screen to be added to the settings page for the specific account
|
|
* type as specified in the account-authenticator.
|
|
@@ -83,6 +89,7 @@ public PreferenceScreen addPreferencesForType(final String accountType,
|
|
try {
|
|
desc = mAuthenticatorHelper.getAccountTypeDescription(accountType);
|
|
if (desc != null && desc.accountPreferencesId != 0) {
|
|
+ Set<String> fragmentAllowList = generateFragmentAllowlist(parent);
|
|
// Load the context of the target package, then apply the
|
|
// base Settings theme (no references to local resources)
|
|
// and create a context theme wrapper so that we get the
|
|
@@ -98,6 +105,12 @@ public PreferenceScreen addPreferencesForType(final String accountType,
|
|
themedCtx.getTheme().setTo(baseTheme);
|
|
prefs = mFragment.getPreferenceManager().inflateFromResource(themedCtx,
|
|
desc.accountPreferencesId, parent);
|
|
+ // Ignore Fragments provided dynamically, as these are coming from external
|
|
+ // applications which must not have access to internal Settings' fragments.
|
|
+ // These preferences are rendered into Settings, so they also won't have access
|
|
+ // to their own Fragments, meaning there is no acceptable usage of
|
|
+ // android:fragment here.
|
|
+ filterBlockedFragments(prefs, fragmentAllowList);
|
|
}
|
|
} catch (PackageManager.NameNotFoundException e) {
|
|
Log.w(TAG, "Couldn't load preferences.xml file from " + desc.packageName);
|
|
@@ -182,6 +195,48 @@ public boolean onPreferenceClick(Preference preference) {
|
|
}
|
|
}
|
|
|
|
+ // Build allowlist from existing Fragments in PreferenceGroup
|
|
+ @VisibleForTesting
|
|
+ Set<String> generateFragmentAllowlist(@Nullable PreferenceGroup prefs) {
|
|
+ Set<String> fragmentAllowList = new ArraySet<>();
|
|
+ if (prefs == null) {
|
|
+ return fragmentAllowList;
|
|
+ }
|
|
+
|
|
+ for (int i = 0; i < prefs.getPreferenceCount(); i++) {
|
|
+ Preference pref = prefs.getPreference(i);
|
|
+ if (pref instanceof PreferenceGroup) {
|
|
+ fragmentAllowList.addAll(generateFragmentAllowlist((PreferenceGroup) pref));
|
|
+ }
|
|
+
|
|
+ String fragmentName = pref.getFragment();
|
|
+ if (!TextUtils.isEmpty(fragmentName)) {
|
|
+ fragmentAllowList.add(fragmentName);
|
|
+ }
|
|
+ }
|
|
+ return fragmentAllowList;
|
|
+ }
|
|
+
|
|
+ // Block clicks on any Preference with android:fragment that is not contained in the allowlist
|
|
+ @VisibleForTesting
|
|
+ void filterBlockedFragments(@Nullable PreferenceGroup prefs,
|
|
+ @NonNull Set<String> allowedFragments) {
|
|
+ if (prefs == null) {
|
|
+ return;
|
|
+ }
|
|
+ for (int i = 0; i < prefs.getPreferenceCount(); i++) {
|
|
+ Preference pref = prefs.getPreference(i);
|
|
+ if (pref instanceof PreferenceGroup) {
|
|
+ filterBlockedFragments((PreferenceGroup) pref, allowedFragments);
|
|
+ }
|
|
+
|
|
+ String fragmentName = pref.getFragment();
|
|
+ if (fragmentName != null && !allowedFragments.contains(fragmentName)) {
|
|
+ pref.setOnPreferenceClickListener(preference -> true);
|
|
+ }
|
|
+ }
|
|
+ }
|
|
+
|
|
/**
|
|
* Determines if the supplied Intent is safe. A safe intent is one that is
|
|
* will launch a exported=true activity or owned by the same uid as the
|