diff options
author | 2023-05-10 03:23:17 +0000 | |
---|---|---|
committer | 2023-05-17 18:19:59 +0000 | |
commit | f4e1d98c751a1c07005d4ba31c4b6de9ddb7958f (patch) | |
tree | 5e5bad7a0298b65ef4e215b27f1263d8858d58b7 | |
parent | 5ea15609bb3030646333231c8e99069a394cfe3a (diff) |
Baseline remaining SimpleManualPermissionEnforcement lint errors
The subject linter catches cases that would be trivial to
migrate from manual permission checks to the @EnforcePermission
annotation. The linter will soon raise an error in the
errorprone build if such a change would be completely
behavior preserving, and thus the manual permission check
should be replaced by the annotation.
Baseline the remaining sites that would raise such an error
so that the linter can be updated accordingly.
Bug: 265014041
Test: TH, manually tested with local changes to linter
Change-Id: I7932f75b2dbd96944fee63d36e15529df3219d3a
-rw-r--r-- | services/accessibility/lint-baseline.xml | 26 | ||||
-rw-r--r-- | services/backup/lint-baseline.xml | 24 | ||||
-rw-r--r-- | services/core/lint-baseline.xml | 8 | ||||
-rw-r--r-- | services/lint-baseline.xml | 59 | ||||
-rw-r--r-- | services/print/lint-baseline.xml | 15 |
5 files changed, 132 insertions, 0 deletions
diff --git a/services/accessibility/lint-baseline.xml b/services/accessibility/lint-baseline.xml new file mode 100644 index 000000000000..6bec8cf5f018 --- /dev/null +++ b/services/accessibility/lint-baseline.xml @@ -0,0 +1,26 @@ +<?xml version="1.0" encoding="UTF-8"?> +<issues format="6" by="lint 8.1.0-dev" type="baseline" client="" dependencies="true" name="" variant="all" version="8.1.0-dev"> + + <issue + id="SimpleManualPermissionEnforcement" + message="IAccessibilityManager permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(" + errorLine2=" ^"> + <location + file="frameworks/base/services/accessibility/java/com/android/server/accessibility/AccessibilityManagerService.java" + line="3992" + column="9"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="IAccessibilityManager permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(" + errorLine2=" ^"> + <location + file="frameworks/base/services/accessibility/java/com/android/server/accessibility/AccessibilityManagerService.java" + line="4007" + column="9"/> + </issue> + +</issues> diff --git a/services/backup/lint-baseline.xml b/services/backup/lint-baseline.xml index 28bb937cfd9c..93c9390feb9c 100644 --- a/services/backup/lint-baseline.xml +++ b/services/backup/lint-baseline.xml @@ -12,4 +12,28 @@ column="16"/> </issue> + <issue + id="SimpleManualPermissionEnforcement" + message="IRestoreSession permission check should be converted to @EnforcePermission annotation"> + <location + file="frameworks/base/services/backup/java/com/android/server/backup/restore/ActiveRestoreSession.java" + line="88"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="IRestoreSession permission check should be converted to @EnforcePermission annotation"> + <location + file="frameworks/base/services/backup/java/com/android/server/backup/restore/ActiveRestoreSession.java" + line="144"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="IRestoreSession permission check should be converted to @EnforcePermission annotation"> + <location + file="frameworks/base/services/backup/java/com/android/server/backup/restore/ActiveRestoreSession.java" + line="207"/> + </issue> + </issues> diff --git a/services/core/lint-baseline.xml b/services/core/lint-baseline.xml index 69e13b38873a..070bd4b1c5a9 100644 --- a/services/core/lint-baseline.xml +++ b/services/core/lint-baseline.xml @@ -137,4 +137,12 @@ line="1448"/> </issue> + <issue + id="SimpleManualPermissionEnforcement" + message="IWindowManager permission check should be converted to @EnforcePermission annotation"> + <location + file="out/.intermediates/frameworks/base/services/core/services.core.protologsrc/gen/services.core.protolog.srcjar!/frameworks/base/services/core/java/com/android/server/wm/WindowManagerService.java" + line="7158"/> + </issue> + </issues> diff --git a/services/lint-baseline.xml b/services/lint-baseline.xml new file mode 100644 index 000000000000..8489c17dd878 --- /dev/null +++ b/services/lint-baseline.xml @@ -0,0 +1,59 @@ +<?xml version="1.0" encoding="UTF-8"?> +<issues format="6" by="lint 8.1.0-dev" type="baseline" client="" dependencies="true" name="" variant="all" version="8.1.0-dev"> + + <issue + id="SimpleManualPermissionEnforcement" + message="ISystemConfig permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(Manifest.permission.READ_CARRIER_APP_INFO," + errorLine2=" ^"> + <location + file="frameworks/base/services/java/com/android/server/SystemConfigService.java" + line="46" + column="13"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="ISystemConfig permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(Manifest.permission.READ_CARRIER_APP_INFO," + errorLine2=" ^"> + <location + file="frameworks/base/services/java/com/android/server/SystemConfigService.java" + line="54" + column="13"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="ISystemConfig permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(Manifest.permission.READ_CARRIER_APP_INFO," + errorLine2=" ^"> + <location + file="frameworks/base/services/java/com/android/server/SystemConfigService.java" + line="67" + column="13"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="ISystemConfig permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(Manifest.permission.GET_RUNTIME_PERMISSIONS," + errorLine2=" ^"> + <location + file="frameworks/base/services/java/com/android/server/SystemConfigService.java" + line="76" + column="13"/> + </issue> + + <issue + id="SimpleManualPermissionEnforcement" + message="ISystemConfig permission check should be converted to @EnforcePermission annotation" + errorLine1=" getContext().enforceCallingOrSelfPermission(Manifest.permission.QUERY_ALL_PACKAGES," + errorLine2=" ^"> + <location + file="frameworks/base/services/java/com/android/server/SystemConfigService.java" + line="107" + column="13"/> + </issue> + +</issues> diff --git a/services/print/lint-baseline.xml b/services/print/lint-baseline.xml new file mode 100644 index 000000000000..1bf031a9e289 --- /dev/null +++ b/services/print/lint-baseline.xml @@ -0,0 +1,15 @@ +<?xml version="1.0" encoding="UTF-8"?> +<issues format="6" by="lint 8.1.0-dev" type="baseline" client="" dependencies="true" name="" variant="all" version="8.1.0-dev"> + + <issue + id="SimpleManualPermissionEnforcement" + message="IPrintManager permission check should be converted to @EnforcePermission annotation" + errorLine1=" mContext.enforceCallingOrSelfPermission(" + errorLine2=" ^"> + <location + file="frameworks/base/services/print/java/com/android/server/print/PrintManagerService.java" + line="401" + column="13"/> + </issue> + +</issues> |