| commit | 5579950a80ef49ea5a54694af147413215b1735d | [log] [tgz] |
|---|---|---|
| author | Stanislav Zholnin <zholnin@google.com> | Fri Mar 08 14:54:55 2019 +0000 |
| committer | Stanislav Zholnin <zholnin@google.com> | Mon Mar 25 11:14:07 2019 +0000 |
| tree | de3d492b413325309ee4273481f21812099c5b29 | |
| parent | 230904927ea921c80b73c34f71df0ffeb93f1110 [diff] [blame] |
Add READ_DEVICE_CONFIG permission check to DeviceConfig API.
Test: atest FrameworksCoreTests:DeviceConfigTest
atest FrameworksCoreTests:SettingsProviderTest
atest SettingsProviderTest:DeviceConfigServiceTest
Bug:117663715
Change-Id: I04226876ddf910945bf343d25fa3dd04ba8eab31
diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java index 7337cdb..e8d726e 100644 --- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java +++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
@@ -1074,8 +1074,7 @@ Slog.v(LOG_TAG, "getConfigSetting(" + name + ")"); } - // TODO(b/117663715): Ensure the caller can access the setting. - // enforceReadPermission(READ_DEVICE_CONFIG); + DeviceConfig.enforceReadPermission(getContext(), /*namespace=*/name.split("/")[0]); // Get the value. synchronized (mLock) {