diff options
author | Brandon Maxwell <maxwelb@google.com> | 2016-03-16 16:51:56 -0700 |
---|---|---|
committer | Brandon Maxwell <maxwelb@google.com> | 2016-03-16 17:09:16 -0700 |
commit | fb7ff8268c1c03837a0f2bf2f2cb51326ebfeeb6 (patch) | |
tree | b7e5fa2032867b84a13b830537195c1390b8cfce | |
parent | fb2993167fcb6315c95e9925a1526c36b0bc0602 (diff) |
Fixing securityException issue
+ This CL ensures that when we're checking if a number is voicemail,
we first check if we have the phone state permission. This is
required to avoid Telecom throwing a SecurityException.
Bug: 27062147
Change-Id: Iee88e01ca8c116e779134dba63d7f8dd6094ac4a
-rw-r--r-- | InCallUI/src/com/android/incallui/CallerInfoUtils.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/InCallUI/src/com/android/incallui/CallerInfoUtils.java b/InCallUI/src/com/android/incallui/CallerInfoUtils.java index aff39566d..289b652fc 100644 --- a/InCallUI/src/com/android/incallui/CallerInfoUtils.java +++ b/InCallUI/src/com/android/incallui/CallerInfoUtils.java @@ -9,13 +9,13 @@ import android.telecom.TelecomManager; import android.text.TextUtils; import android.util.Log; -import com.android.contacts.common.compat.telecom.TelecomManagerCompat; import com.android.contacts.common.model.Contact; import com.android.contacts.common.model.ContactLoader; import com.android.dialer.R; import com.android.dialer.calllog.ContactInfo; import com.android.dialer.service.CachedNumberLookupService; import com.android.dialer.service.CachedNumberLookupService.CachedContactInfo; +import com.android.dialer.util.TelecomUtil; import java.util.Arrays; @@ -118,8 +118,7 @@ public class CallerInfoUtils { } public static boolean isVoiceMailNumber(Context context, Call call) { - return TelecomManagerCompat.isVoiceMailNumber( - (TelecomManager) context.getSystemService(Context.TELECOM_SERVICE), + return TelecomUtil.isVoicemailNumber(context, call.getTelecomCall().getDetails().getAccountHandle(), call.getNumber()); } |