From 89c9ff500a3af31740e5cf082e30b79c9cc8451e Mon Sep 17 00:00:00 2001
From: twyen <twyen@google.com>
Date: Thu, 14 Dec 2017 17:26:58 -0800
Subject: 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
---
 java/com/android/dialer/telecom/TelecomUtil.java | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'java/com/android/dialer/telecom')

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) {
-- 
cgit v1.2.3