diff options
| author | Michael Groover <mpgroover@google.com> | 2020-06-20 15:25:53 -0700 |
|---|---|---|
| committer | Michael Groover <mpgroover@google.com> | 2020-06-21 16:47:58 -0700 |
| commit | 8d7d92d71055e7740fe1ba993960774070aea6f5 (patch) | |
| tree | 53dcb2814792cbfca68247b6fa92c7a5a1afa2aa /tools/aapt2/java/AnnotationProcessor.cpp | |
| parent | a27465258acbc7e4f0007cf2ab3d0cbfd1294893 (diff) | |
Reorder TelephonyPermissions calls for carrier privileges
The SubscriptionManager APIs that return SubscriptionInfo objects are
often invoked by carrier privileged apps to obtain the details about
the subscription(s). Identifier and phone number access checks
currently verify requirements that typically cannot be satisfied by
carrier privileged apps first before verifying an app holds carrier
privileges. This commit invokes the carrier privileges check before
the generic PermissionManagerService check for identifier access
and moves the WRITE_SMS appop check after the READ_PHONE_STATE /
carrier privilege check for phone number access.
Bug: 157642567
Fixes: 73308711
Test: atest TelephonyPermissionsTest
Test: atest SubscriptionControllerTest
Change-Id: I0a446af5c2adaf1d6b06da221f9e236b1bdde146
Diffstat (limited to 'tools/aapt2/java/AnnotationProcessor.cpp')
0 files changed, 0 insertions, 0 deletions