Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[SettingsProvider] fix server-side Readable maxTargetSdk check

Browse Source 
I must have used the wrong autocomplete from Intellij. Previous tests were not able to catch this because we have the same check on the client side.

BUG: 172838801
BUG: 211711793
Test: atest android.appsecurity.cts.ReadableSettingsFieldsTest
Change-Id: Ia3e4541564d9928ff04fa95564bb9e043d49e8ee
Merged-In: Ia3e4541564d9928ff04fa95564bb9e043d49e8ee
This commit is contained in:
Songchun Fan
2021-12-28 19:40:59 +00:00
parent 86643bc711
commit 5568cc227e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
View File

@@ -2102,7 +2102,7 @@ public class SettingsProvider extends ContentProvider {
}
if ((ai.flags & ApplicationInfo.FLAG_TEST_ONLY) == 0) {
// Skip checking readable annotations for test_only apps
checkReadableAnnotation(settingsType, settingName, ai.targetSandboxVersion);
checkReadableAnnotation(settingsType, settingName, ai.targetSdkVersion);
}
/**
* some settings need additional permission check, this is to have a matching security