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] |
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) {