Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8ac3494f authored by arunvoddu's avatar arunvoddu
Browse files

Changed permission from READ_PHONE_STATE to READ_BASIC_PHONE_SATE for TM API. 

TM#getCarrierRestrictionStatus permissions changes in order to avoid facing the
securityException in case the user disabled the READ_PHONE_STATE.

Flag: EXEMPT bugfix
Bug: 359910900
Test: CTS verified
Change-Id: Iddc8591f03bfaa002238021c26fa3d6476ecd25b
parent d3c41192

core/api/current.txt

+1 −1
Original line number Diff line number Diff line
@@ -46067,7 +46067,7 @@ package android.telephony { 
    method public int getCardIdForDefaultEuicc();

    method @RequiresPermission(android.Manifest.permission.READ_PHONE_STATE) @WorkerThread public android.os.PersistableBundle getCarrierConfig();

    method public int getCarrierIdFromSimMccMnc();

    method @RequiresPermission(android.Manifest.permission.READ_PHONE_STATE) public void getCarrierRestrictionStatus(@NonNull java.util.concurrent.Executor, @NonNull java.util.function.Consumer<java.lang.Integer>);

    method @RequiresPermission(anyOf={android.Manifest.permission.READ_BASIC_PHONE_STATE, android.Manifest.permission.READ_PHONE_STATE}) public void getCarrierRestrictionStatus(@NonNull java.util.concurrent.Executor, @NonNull java.util.function.Consumer<java.lang.Integer>);

    method @Deprecated @RequiresPermission(android.Manifest.permission.ACCESS_FINE_LOCATION) public android.telephony.CellLocation getCellLocation();

    method public int getDataActivity();

    method @RequiresPermission(anyOf={android.Manifest.permission.READ_PHONE_STATE, android.Manifest.permission.READ_BASIC_PHONE_STATE}) public int getDataNetworkType();

telephony/java/android/telephony/TelephonyManager.java

+4 −1
Original line number Diff line number Diff line
@@ -13928,7 +13928,10 @@ public class TelephonyManager { 
     *          {@link PackageManager#FEATURE_TELEPHONY_SUBSCRIPTION}.

     */

    @RequiresFeature(PackageManager.FEATURE_TELEPHONY_SUBSCRIPTION)

    @RequiresPermission(android.Manifest.permission.READ_PHONE_STATE)

    @RequiresPermission(anyOf = {

            android.Manifest.permission.READ_BASIC_PHONE_STATE,

            android.Manifest.permission.READ_PHONE_STATE

    })

    public void getCarrierRestrictionStatus(@NonNull Executor executor,

            @NonNull @CarrierRestrictionStatus

                    Consumer<Integer> resultListener) {