From ab384c5b7210d35ebf88105859d353c7dad56d99 Mon Sep 17 00:00:00 2001 From: Hassan Ali Date: Mon, 28 Nov 2022 12:44:52 +0000 Subject: [PATCH] Stop calling Enforcereadpermission from DeviceConf Stop calling enforcereadpermission from DeviceConfig.java because It's not safe to call Enforcereadpermission from the client peocess side. Test: m Bug: 258220607 Change-Id: I9e83866f41ebe174888722336c45cf0986359e91 --- core/api/system-current.txt | 2 +- core/java/android/provider/DeviceConfig.java | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/core/api/system-current.txt b/core/api/system-current.txt index 5e86546b56db6..01ecc51c8e2e9 100644 --- a/core/api/system-current.txt +++ b/core/api/system-current.txt @@ -10529,7 +10529,7 @@ package android.provider { } public final class DeviceConfig { - method @RequiresPermission(android.Manifest.permission.READ_DEVICE_CONFIG) public static void addOnPropertiesChangedListener(@NonNull String, @NonNull java.util.concurrent.Executor, @NonNull android.provider.DeviceConfig.OnPropertiesChangedListener); + method public static void addOnPropertiesChangedListener(@NonNull String, @NonNull java.util.concurrent.Executor, @NonNull android.provider.DeviceConfig.OnPropertiesChangedListener); method @RequiresPermission(android.Manifest.permission.WRITE_DEVICE_CONFIG) public static boolean deleteProperty(@NonNull String, @NonNull String); method @RequiresPermission(android.Manifest.permission.READ_DEVICE_CONFIG) public static boolean getBoolean(@NonNull String, @NonNull String, boolean); method @RequiresPermission(android.Manifest.permission.READ_DEVICE_CONFIG) public static float getFloat(@NonNull String, @NonNull String, float); diff --git a/core/java/android/provider/DeviceConfig.java b/core/java/android/provider/DeviceConfig.java index be110ebb92ea2..7df9290274ce9 100644 --- a/core/java/android/provider/DeviceConfig.java +++ b/core/java/android/provider/DeviceConfig.java @@ -1167,12 +1167,10 @@ public final class DeviceConfig { * @see #removeOnPropertiesChangedListener(OnPropertiesChangedListener) */ @SystemApi - @RequiresPermission(READ_DEVICE_CONFIG) public static void addOnPropertiesChangedListener( @NonNull String namespace, @NonNull @CallbackExecutor Executor executor, @NonNull OnPropertiesChangedListener onPropertiesChangedListener) { - Settings.Config.enforceReadPermission(namespace); synchronized (sLock) { Pair oldNamespace = sListeners.get(onPropertiesChangedListener); if (oldNamespace == null) {