summaryrefslogtreecommitdiff
path: root/services
diff options
context:
space:
mode:
author mattgilbride <mattgilbride@google.com> 2023-05-10 03:23:17 +0000
committer mattgilbride <mattgilbride@google.com> 2023-05-17 18:19:59 +0000
commitf4e1d98c751a1c07005d4ba31c4b6de9ddb7958f (patch)
tree5e5bad7a0298b65ef4e215b27f1263d8858d58b7 /services
parent5ea15609bb3030646333231c8e99069a394cfe3a (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
Diffstat (limited to 'services')
-rw-r--r--services/accessibility/lint-baseline.xml26
-rw-r--r--services/backup/lint-baseline.xml24
-rw-r--r--services/core/lint-baseline.xml8
-rw-r--r--services/lint-baseline.xml59
-rw-r--r--services/print/lint-baseline.xml15
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>