summaryrefslogtreecommitdiff
path: root/native/android/configuration.cpp
diff options
context:
space:
mode:
author Aishwarya Mallampati <amallampati@google.com> 2022-11-01 17:04:35 +0000
committer Aishwarya Mallampati <amallampati@google.com> 2023-05-25 18:41:17 +0000
commite46bce078fef9dba500a7411e843f7f00a7a33c2 (patch)
tree165d350b15cddc5554ddda16fccd070f3a2cce7d /native/android/configuration.cpp
parent850fd984e5f346645b5a941ed7307387c7e4c4de (diff)
DO NOT MERGE Grant carrier privileges if package has carrier config access.
TelephonyManager#hasCarrierPrivileges internally uses SubscriptionManager#canManageSubscription to decide whether to grant carrier privilege status to an app or not. SubscriptionManager#canManageSubscription returns true if caller APK's certificate matches with one of the mNativeAccessRules or mCarrierConfigAccessRules. This over-grants carrier privilege status to apps that only has mNativeAccessRules. Carrier privilege status should be granted to the caller APK only if it's certificate matches with one of mCarrierConfigAccessRules. Replaced SubscriptionManager#canManageSubscription with PhoneInterfaceManager#hasCarrierConfigAccess which returns true only if caller APK certificates matches with one of mCarrierConfigAccessRules of the given subscription. Bug: 226593252 Test: Manual Testing as explained in b/226593252#comment51 atest CtsTelephonyTestCases Flashed build on raven-userdebug and performed basic funtionality tests Change-Id: I662064529d2a9348f395fe3b541366de8bc2fe7d
Diffstat (limited to 'native/android/configuration.cpp')
0 files changed, 0 insertions, 0 deletions