commit d2c96928181af43c2e93f1236e4f8f251b1cfb9e
author Yorke Lee <yorkelee@google.com> Fri Sep 18 12:59:16 2015 -0700
committer Yorke Lee <yorkelee@google.com> Fri Sep 18 15:04:42 2015 -0700
tree 9f54a7f99a1cdd02c088c152427b1a91b968c73f
parent 4ae9ca38a4fc9d25c50935f449a87289841685ee

Protect CallUtil methods with permission check

Bug: 20266292
Change-Id: Id19527670876bd4659d2e853b76ec6496986036b

src/com/android/contacts/common/CallUtil.java

diff --git a/src/com/android/contacts/common/CallUtil.java b/src/com/android/contacts/common/CallUtil.java
index 1b998b5..041b1b4 100644
--- a/src/com/android/contacts/common/CallUtil.java
+++ b/src/com/android/contacts/common/CallUtil.java
@@ -16,6 +16,7 @@
 
 package com.android.contacts.common;
 
+import com.android.contacts.common.util.PermissionsUtil;
 import com.android.contacts.common.util.PhoneNumberHelper;
 import com.android.phone.common.PhoneConstants;
 
@@ -108,6 +109,9 @@
      *      {@code false} otherwise.
      */
     public static boolean isVideoEnabled(Context context) {
+        if (!PermissionsUtil.hasPermission(context, android.Manifest.permission.READ_PHONE_STATE)) {
+            return false;
+        }
         TelecomManager telecommMgr = (TelecomManager)
                 context.getSystemService(Context.TELECOM_SERVICE);
         if (telecommMgr == null) {
@@ -133,6 +137,9 @@
      *      subject specified, {@code false} otherwise.
      */
     public static boolean isCallWithSubjectSupported(Context context) {
+        if (!PermissionsUtil.hasPermission(context, android.Manifest.permission.READ_PHONE_STATE)) {
+            return false;
+        }
         TelecomManager telecommMgr = (TelecomManager)
                 context.getSystemService(Context.TELECOM_SERVICE);
         if (telecommMgr == null) {