diff options
| author | 2019-03-13 14:43:26 +0000 | |
|---|---|---|
| committer | 2019-03-13 14:43:26 +0000 | |
| commit | 5a25a84b9efee61787c90deacc3ceb2aedb4fafe (patch) | |
| tree | f8bc3fb42294b604f6773db3086feabed6c5fb31 | |
| parent | c7f498fe0bceca283c384ac303422fd8e9d0eccf (diff) | |
Add WRITE_DEVICE_CONFIG permission check to DeviceConfig API.
Test: atest FrameworksCoreTests:DeviceConfigTest
atest FrameworksCoreTests:SettingsProviderTest
atest SettingsProviderTest:DeviceConfigServiceTest
Bug:117663715
Change-Id: Iab7dd7959891ebc29fbbf773ce6ab28ba4e05f1d
| -rw-r--r-- | packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java index 2c2987c88077..ff25ad9f037b 100644 --- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java +++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java @@ -1103,9 +1103,7 @@ public class SettingsProvider extends ContentProvider { private boolean mutateConfigSetting(String name, String value, String prefix, boolean makeDefault, int operation, int mode) { - - // TODO(b/117663715): Ensure the caller can access the setting. - // enforceReadPermission(WRITE_DEVICE_CONFIG); + enforceWritePermission(Manifest.permission.WRITE_DEVICE_CONFIG); // Perform the mutation. synchronized (mLock) { |