commit 051ad42cdc9e577a602a3eca33e288ed6add16e6
author Pranav Madapurmath <pmadapurmath@google.com> Mon Feb 05 15:09:57 2024 -0800
committer Pranav Madapurmath <pmadapurmath@google.com> Wed Feb 07 22:59:17 2024 +0000
tree d7a19d71e724e5136d5d19341d146856322b31cb
parent ac635e24ed031e15cd7f068d0fcf90b6f59d275d

Formalize EXTRA_SKIP_CALL_FILTERING as system API

As part of telecom mainline prep, resolve the hidden reference from wear
services to EXTRA_SKIP_CALL_FILTERING by promoting it to a system API.

Bug: 322518197
Bug: 311773409
Test: atest PhoneAccountOperationsTest

Change-Id: Id96460b6040a47b83106e0d32fbd6ec74ed20a07

src/com/android/server/telecom/TelecomServiceImpl.java

diff --git a/src/com/android/server/telecom/TelecomServiceImpl.java b/src/com/android/server/telecom/TelecomServiceImpl.java
index 61fa9ba..313a164 100644
--- a/src/com/android/server/telecom/TelecomServiceImpl.java
+++ b/src/com/android/server/telecom/TelecomServiceImpl.java
@@ -787,6 +787,9 @@
                         Bundle extras = account.getExtras();
                         if (extras != null
                                 && extras.getBoolean(PhoneAccount.EXTRA_SKIP_CALL_FILTERING)) {
+                            // System apps should be granted the MODIFY_PHONE_STATE permission.
+                            enforceModifyPermission(
+                                    "registerPhoneAccount requires MODIFY_PHONE_STATE permission.");
                             enforceRegisterSkipCallFiltering();
                         }
                         final int callingUid = Binder.getCallingUid();