commit 72a22764d4e8df2d1ae19d02e2fa363d93984a0e
author Sooraj Sasindran <sasindran@google.com> Mon Nov 08 12:01:16 2021 -0800
committer Sooraj Sasindran <sasindran@google.com> Wed Dec 01 07:42:23 2021 +0000
tree f4c46b5781224498ba31710b61faf6af6d397c37
parent 191552ddccb94f41039b5f593316294a5e80542b

Verify carrier privilege for isDataEnabledForReason

Check for carrier privilege and modify phone state permission
for isDataEnabledForReason api

Bug: 205599503
Test: build
Merged-In: Ied669505b3558e2ad1d66f95311f89bb0fa6bbe9
Change-Id: Ied669505b3558e2ad1d66f95311f89bb0fa6bbe9

src/com/android/phone/PhoneInterfaceManager.java

diff --git a/src/com/android/phone/PhoneInterfaceManager.java b/src/com/android/phone/PhoneInterfaceManager.java
index a0b2277..4eae7ec 100755
--- a/src/com/android/phone/PhoneInterfaceManager.java
+++ b/src/com/android/phone/PhoneInterfaceManager.java
@@ -6663,8 +6663,13 @@
             mApp.enforceCallingOrSelfPermission(android.Manifest.permission.ACCESS_NETWORK_STATE,
                     null);
         } catch (Exception e) {
-            mApp.enforceCallingOrSelfPermission(android.Manifest.permission.READ_PHONE_STATE,
-                    "isDataEnabledForReason");
+            try {
+                mApp.enforceCallingOrSelfPermission(android.Manifest.permission.READ_PHONE_STATE,
+                        "isDataEnabledForReason");
+            } catch (Exception e2) {
+                TelephonyPermissions.enforceCallingOrSelfModifyPermissionOrCarrierPrivilege(
+                        mApp, subId, "isDataEnabledForReason");
+            }
         }