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

Commit 8fe1f7cb authored by Nazanin Bakhshi's avatar Nazanin Bakhshi Committed by Malcolm Chen
Browse files

add checking for carrierConfigs certificates for psim 

Test: utests
Bug: 139133814
Change-Id: I2d5cf7ee4e516ff16a67a5fbef6674a380095888
Merged-In: I2d5cf7ee4e516ff16a67a5fbef6674a380095888
parent 398794c0

telephony/java/android/telephony/SubscriptionInfo.java

+0 −11
Original line number Diff line number Diff line
@@ -551,7 +551,6 @@ public class SubscriptionInfo implements Parcelable { 
     *

     * @param context Context of the application to check.

     * @return whether the app is authorized to manage this subscription per its metadata.

     * @throws UnsupportedOperationException if this subscription is not embedded.

     * @hide

     * @deprecated - Do not use.

     */
@@ -567,15 +566,11 @@ public class SubscriptionInfo implements Parcelable { 
     * @param context Any context.

     * @param packageName Package name of the app to check.

     * @return whether the app is authorized to manage this subscription per its metadata.

     * @throws UnsupportedOperationException if this subscription is not embedded.

     * @hide

     * @deprecated - Do not use.

     */

    @Deprecated

    public boolean canManageSubscription(Context context, String packageName) {

        if (!isEmbedded()) {

            throw new UnsupportedOperationException("Not an embedded subscription");

        }

        List<UiccAccessRule> allAccessRules = getAllAccessRules();

        if (allAccessRules == null) {

            return false;
@@ -606,9 +601,6 @@ public class SubscriptionInfo implements Parcelable { 
     */

    @SystemApi

    public @Nullable List<UiccAccessRule> getAccessRules() {

        if (!isEmbedded()) {

            throw new UnsupportedOperationException("Not an embedded subscription");

        }

        if (mNativeAccessRules == null) return null;

        return Arrays.asList(mNativeAccessRules);

    }
@@ -619,9 +611,6 @@ public class SubscriptionInfo implements Parcelable { 
     * @hide

     */

    public @Nullable List<UiccAccessRule> getAllAccessRules() {

        if (!isEmbedded()) {

            throw new UnsupportedOperationException("Not an embedded subscription");

        }

        List<UiccAccessRule> merged = new ArrayList<>();

        if (mNativeAccessRules != null) merged.addAll(getAccessRules());

        if (mCarrierConfigAccessRules != null) {

telephony/java/android/telephony/SubscriptionManager.java

+1 −6
Original line number Diff line number Diff line
@@ -2624,7 +2624,6 @@ public class SubscriptionManager { 
     *

     * @param info The subscription to check.

     * @return whether the app is authorized to manage this subscription per its metadata.

     * @throws IllegalArgumentException if this subscription is not embedded.

     */

    public boolean canManageSubscription(SubscriptionInfo info) {

        return canManageSubscription(info, mContext.getPackageName());
@@ -2640,13 +2639,9 @@ public class SubscriptionManager { 
     * @param info The subscription to check.

     * @param packageName Package name of the app to check.

     * @return whether the app is authorized to manage this subscription per its access rules.

     * @throws IllegalArgumentException if this subscription is not embedded.

     * @hide

     */

    public boolean canManageSubscription(SubscriptionInfo info, String packageName) {

        if (!info.isEmbedded()) {

            throw new IllegalArgumentException("Not an embedded subscription");

        }

        if (info.getAllAccessRules() == null) {

            return false;

        }
@@ -3048,7 +3043,7 @@ public class SubscriptionManager { 
        // to the caller.

        boolean hasCarrierPrivilegePermission = TelephonyManager.from(mContext)

                .hasCarrierPrivileges(info.getSubscriptionId())

                || (info.isEmbedded() && canManageSubscription(info));

                || canManageSubscription(info);

        return hasCarrierPrivilegePermission;

    }