summaryrefslogtreecommitdiff
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
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
-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>