diff options
author | twyen <twyen@google.com> | 2017-12-14 17:26:58 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2017-12-14 17:44:38 -0800 |
commit | 89c9ff500a3af31740e5cf082e30b79c9cc8451e (patch) | |
tree | d4d7610b759a27ffa35682dbd3a9d4b60aff4662 /java/com/android/dialer/telecom | |
parent | 937bd02d7cb045cea9aa21a76f3425a68468eeeb (diff) |
Add PreCall step to check permission
If CALL_PHONE is missing PreCall will abort immediately and show a toast.
Bug: 70565989,70566029
Test: PermissionCheckActionTest
PiperOrigin-RevId: 179123070
Change-Id: I0d43c6acd44e77562752c871bed0223b2f787dde
Diffstat (limited to 'java/com/android/dialer/telecom')
-rw-r--r-- | java/com/android/dialer/telecom/TelecomUtil.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/java/com/android/dialer/telecom/TelecomUtil.java b/java/com/android/dialer/telecom/TelecomUtil.java index 6f424de4a..22f3727e6 100644 --- a/java/com/android/dialer/telecom/TelecomUtil.java +++ b/java/com/android/dialer/telecom/TelecomUtil.java @@ -157,6 +157,9 @@ public abstract class TelecomUtil { if (TextUtils.isEmpty(phoneAccountHandle.getId())) { return Optional.absent(); } + if (!hasPermission(context, permission.READ_PHONE_STATE)) { + return Optional.absent(); + } SubscriptionManager subscriptionManager = context.getSystemService(SubscriptionManager.class); List<SubscriptionInfo> subscriptionInfos = subscriptionManager.getActiveSubscriptionInfoList(); if (subscriptionInfos == null) { |