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

Commit b2c75c45 authored by Meng Wang's avatar Meng Wang Committed by Gerrit Code Review
Browse files

Merge changes from topic "phonenumber" 

* changes:
  Populate IMS phone number
  Add getPhoneNumber/setPhoneNumber API
parents de7bc68e 4fc60313

src/java/com/android/internal/telephony/SubscriptionController.java

+106 −0
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ 


package com.android.internal.telephony;



import static android.Manifest.permission.READ_PHONE_NUMBERS;

import static android.Manifest.permission.READ_PRIVILEGED_PHONE_STATE;

import static android.content.pm.PackageManager.PERMISSION_GRANTED;

import static android.telephony.TelephonyManager.MULTISIM_ALLOWED;

import static android.telephony.TelephonyManager.SET_OPPORTUNISTIC_SUB_REMOTE_SERVICE_EXCEPTION;
@@ -45,6 +47,7 @@ import android.os.RemoteException; 
import android.os.TelephonyServiceManager.ServiceRegisterer;

import android.os.UserHandle;

import android.provider.Settings;

import android.provider.Telephony.SimInfo;

import android.telecom.PhoneAccountHandle;

import android.telecom.TelecomManager;

import android.telephony.AnomalyReporter;
@@ -3200,6 +3203,8 @@ public class SubscriptionController extends ISub.Stub { 
                value.put(propKey, Integer.parseInt(propValue));

                break;

            case SubscriptionManager.ALLOWED_NETWORK_TYPES:

            case SimInfo.COLUMN_PHONE_NUMBER_SOURCE_CARRIER:

            case SimInfo.COLUMN_PHONE_NUMBER_SOURCE_IMS:

                value.put(propKey, propValue);

                break;

            default:
@@ -3278,6 +3283,8 @@ public class SubscriptionController extends ISub.Stub { 
                        case SubscriptionManager.D2D_STATUS_SHARING:

                        case SubscriptionManager.D2D_STATUS_SHARING_SELECTED_CONTACTS:

                        case SubscriptionManager.NR_ADVANCED_CALLING_ENABLED:

                        case SimInfo.COLUMN_PHONE_NUMBER_SOURCE_CARRIER:

                        case SimInfo.COLUMN_PHONE_NUMBER_SOURCE_IMS:

                            resultValue = cursor.getString(0);

                            break;

                        default:
@@ -4508,6 +4515,105 @@ public class SubscriptionController extends ISub.Stub { 
        }

    }



    /*

     * Returns the phone number for the given {@code subId} and {@code source},

     * or an empty string if not available.

     */

    @Override

    public String getPhoneNumber(int subId, int source,

            String callingPackage, String callingFeatureId) {

        TelephonyPermissions.enforceAnyPermissionGrantedOrCarrierPrivileges(

                mContext, subId, Binder.getCallingUid(), "getPhoneNumber",

                READ_PHONE_NUMBERS, READ_PRIVILEGED_PHONE_STATE);



        final long identity = Binder.clearCallingIdentity();

        try {

            String number = getPhoneNumber(subId, source);

            return number == null ? "" : number;

        } finally {

            Binder.restoreCallingIdentity(identity);

        }

    }



    /*

     * Returns the phone number for the given {@code subId} or an empty string if not available.

     *

     * <p>Built up on getPhoneNumber(int subId, int source) this API picks the 1st available

     * source based on a priority order.

     */

    @Override

    public String getPhoneNumberFromFirstAvailableSource(int subId,

            String callingPackage, String callingFeatureId) {

        TelephonyPermissions.enforceAnyPermissionGrantedOrCarrierPrivileges(

                mContext, subId, Binder.getCallingUid(), "getPhoneNumberFromFirstAvailableSource",

                READ_PHONE_NUMBERS, READ_PRIVILEGED_PHONE_STATE);



        final long identity = Binder.clearCallingIdentity();

        try {

            String numberFromCarrier = getPhoneNumber(

                    subId, SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER);

            if (!TextUtils.isEmpty(numberFromCarrier)) {

                return numberFromCarrier;

            }

            String numberFromUicc = getPhoneNumber(

                    subId, SubscriptionManager.PHONE_NUMBER_SOURCE_UICC);

            if (!TextUtils.isEmpty(numberFromUicc)) {

                return numberFromUicc;

            }

            String numberFromIms = getPhoneNumber(

                    subId, SubscriptionManager.PHONE_NUMBER_SOURCE_IMS);

            if (!TextUtils.isEmpty(numberFromIms)) {

                return numberFromIms;

            }

            return "";

        } finally {

            Binder.restoreCallingIdentity(identity);

        }

    }



    // Internal helper method for implementing getPhoneNumber() API.

    @Nullable

    private String getPhoneNumber(int subId, int source) {

        if (source == SubscriptionManager.PHONE_NUMBER_SOURCE_UICC) {

            Phone phone = PhoneFactory.getPhone(getPhoneId(subId));

            return phone != null ? phone.getLine1Number() : null;

        }

        if (source == SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER) {

            return getSubscriptionProperty(subId, SimInfo.COLUMN_PHONE_NUMBER_SOURCE_CARRIER);

        }

        if (source == SubscriptionManager.PHONE_NUMBER_SOURCE_IMS) {

            return getSubscriptionProperty(subId, SimInfo.COLUMN_PHONE_NUMBER_SOURCE_IMS);

        }

        throw new IllegalArgumentException("setPhoneNumber doesn't accept source " + source);

    }



    /**

     * Sets the phone number for the given {@code subId}.

     *

     * <p>The only accepted {@code source} is {@link

     * SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER}.

     */

    @Override

    public void setPhoneNumber(int subId, int source, String number,

            String callingPackage, String callingFeatureId) {

        if (source != SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER) {

            throw new IllegalArgumentException("setPhoneNumber doesn't accept source " + source);

        }

        if (!TelephonyPermissions.checkCarrierPrivilegeForSubId(mContext, subId)) {

            throw new SecurityException("setPhoneNumber for CARRIER needs carrier privilege");

        }

        if (number == null) {

            throw new NullPointerException("invalid number null");

        }



        final long identity = Binder.clearCallingIdentity();

        try {

            setSubscriptionProperty(subId, SimInfo.COLUMN_PHONE_NUMBER_SOURCE_CARRIER, number);

        } finally {

            Binder.restoreCallingIdentity(identity);

        }

    }



    /**

     * @hide

     */

src/java/com/android/internal/telephony/imsphone/ImsPhone.java

+57 −0
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 


package com.android.internal.telephony.imsphone;



import static android.provider.Telephony.SimInfo.COLUMN_PHONE_NUMBER_SOURCE_IMS;

import static android.telephony.ims.ImsManager.EXTRA_WFC_REGISTRATION_FAILURE_MESSAGE;

import static android.telephony.ims.ImsManager.EXTRA_WFC_REGISTRATION_FAILURE_TITLE;
@@ -70,6 +71,7 @@ import android.telephony.CarrierConfigManager; 
import android.telephony.NetworkRegistrationInfo;

import android.telephony.PhoneNumberUtils;

import android.telephony.ServiceState;

import android.telephony.SubscriptionInfo;

import android.telephony.SubscriptionManager;

import android.telephony.TelephonyManager;

import android.telephony.UssdResponse;
@@ -105,6 +107,7 @@ import com.android.internal.telephony.Phone; 
import com.android.internal.telephony.PhoneConstants;

import com.android.internal.telephony.PhoneNotifier;

import com.android.internal.telephony.ServiceStateTracker;

import com.android.internal.telephony.SubscriptionController;

import com.android.internal.telephony.TelephonyComponentFactory;

import com.android.internal.telephony.TelephonyIntents;

import com.android.internal.telephony.dataconnection.TransportManager;
@@ -122,6 +125,7 @@ import com.android.telephony.Rlog; 
import java.io.FileDescriptor;

import java.io.PrintWriter;

import java.util.ArrayList;

import java.util.Arrays;

import java.util.List;

import java.util.function.Consumer;
@@ -2485,9 +2489,62 @@ public class ImsPhone extends ImsPhoneBase { 
        public void handleImsSubscriberAssociatedUriChanged(Uri[] uris) {

            if (DBG) logd("handleImsSubscriberAssociatedUriChanged");

            setCurrentSubscriberUris(uris);

            setPhoneNumberForSourceIms(uris);

        }

    };



    /** Sets the IMS phone number from IMS associated URIs, if any found. */

    @VisibleForTesting

    public void setPhoneNumberForSourceIms(Uri[] uris) {

        String phoneNumber = extractPhoneNumberFromAssociatedUris(uris);

        if (phoneNumber == null) {

            return;

        }

        SubscriptionController subController = SubscriptionController.getInstance();

        int subId = getSubId();

        String countryIso = getCountryIso(subController, subId);

        // Format the number as one more defense to reject garbage values:

        // phoneNumber will become null.

        phoneNumber = PhoneNumberUtils.formatNumberToE164(phoneNumber, countryIso);

        if (phoneNumber == null) {

            return;

        }

        subController.setSubscriptionProperty(subId, COLUMN_PHONE_NUMBER_SOURCE_IMS, phoneNumber);

    }



    private static String getCountryIso(SubscriptionController subController, int subId) {

        SubscriptionInfo info = subController.getSubscriptionInfo(subId);

        String countryIso = info == null ? "" : info.getCountryIso();

        // info.getCountryIso() may return null

        return countryIso == null ? "" : countryIso;

    }



    /**

     * Finds the phone number from associated URIs.

     *

     * <p>Associated URIs are public user identities, and phone number could be used:

     * see 3GPP TS 24.229 5.4.1.2 and 3GPP TS 23.003 13.4. This algotihm look for the

     * possible "global number" in E.164 format.

     */

    private static String extractPhoneNumberFromAssociatedUris(Uri[] uris) {

        if (uris == null) {

            return null;

        }

        return Arrays.stream(uris)

                // Phone number is an opaque URI "tel:<phone-number>" or "sip:<phone-number>@<...>"

                .filter(u -> u != null && u.isOpaque())

                .filter(u -> "tel".equalsIgnoreCase(u.getScheme())

                        || "sip".equalsIgnoreCase(u.getScheme()))

                .map(Uri::getSchemeSpecificPart)

                // "Global number" should be in E.164 format starting with "+" e.g. "+447539447777"

                .filter(ssp -> ssp != null && ssp.startsWith("+"))

                // Remove whatever after "@" for sip URI

                .map(ssp -> ssp.split("@")[0])

                // Returns the first winner

                .findFirst()

                .orElse(null);

    }



    public IccRecords getIccRecords() {

        return mDefaultPhone.getIccRecords();

    }

tests/telephonytests/src/com/android/internal/telephony/imsphone/ImsPhoneTest.java

+96 −0
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ 


package com.android.internal.telephony.imsphone;



import static android.Manifest.permission.MODIFY_PHONE_STATE;

import static android.provider.Telephony.SimInfo.COLUMN_PHONE_NUMBER_SOURCE_IMS;

import static android.telephony.CarrierConfigManager.USSD_OVER_CS_ONLY;

import static android.telephony.CarrierConfigManager.USSD_OVER_CS_PREFERRED;

import static android.telephony.CarrierConfigManager.USSD_OVER_IMS_ONLY;
@@ -40,6 +42,7 @@ import static org.mockito.Mockito.doNothing; 
import static org.mockito.Mockito.doReturn;

import static org.mockito.Mockito.doThrow;

import static org.mockito.Mockito.eq;

import static org.mockito.Mockito.mock;

import static org.mockito.Mockito.never;

import static org.mockito.Mockito.reset;

import static org.mockito.Mockito.times;
@@ -49,6 +52,7 @@ import android.app.Activity; 
import android.content.BroadcastReceiver;

import android.content.Intent;

import android.content.res.Resources;

import android.net.Uri;

import android.os.AsyncResult;

import android.os.Bundle;

import android.os.Handler;
@@ -57,6 +61,7 @@ import android.os.PersistableBundle; 
import android.sysprop.TelephonyProperties;

import android.telephony.CarrierConfigManager;

import android.telephony.ServiceState;

import android.telephony.SubscriptionInfo;

import android.telephony.TelephonyManager;

import android.telephony.ims.ImsCallProfile;

import android.telephony.ims.ImsReasonInfo;
@@ -976,6 +981,97 @@ public class ImsPhoneTest extends TelephonyTest { 
        mImsPhoneUT.handleMessage(m);

    }



    @Test

    @SmallTest

    public void testSetPhoneNumberForSourceIms() {

        // In reality the method under test runs in phone process so has MODIFY_PHONE_STATE

        mContextFixture.addCallingOrSelfPermission(MODIFY_PHONE_STATE);

        int subId = 1;

        doReturn(subId).when(mPhone).getSubId();

        SubscriptionInfo subInfo = mock(SubscriptionInfo.class);

        doReturn("gb").when(subInfo).getCountryIso();

        doReturn(subInfo).when(mSubscriptionController).getSubscriptionInfo(subId);



        // 1. Two valid phone number; 1st is set.

        Uri[] associatedUris = new Uri[] {

            Uri.parse("sip:+447539447777@ims.x.com"),

            Uri.parse("tel:+447539446666")

        };

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController).setSubscriptionProperty(

                subId, COLUMN_PHONE_NUMBER_SOURCE_IMS, "+447539447777");



        // 2. 1st invalid and 2nd valid: 2nd is set.

        associatedUris = new Uri[] {

            Uri.parse("sip:447539447777@ims.x.com"),

            Uri.parse("tel:+447539446666")

        };

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController).setSubscriptionProperty(

                subId, COLUMN_PHONE_NUMBER_SOURCE_IMS, "+447539446666");



        // 3. 1st sip-uri is not phone number and 2nd valid: 2nd is set.

        associatedUris = new Uri[] {

            Uri.parse("sip:john.doe@ims.x.com"),

            Uri.parse("tel:+447539446677"),

            Uri.parse("sip:+447539447766@ims.x.com")

        };

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController).setSubscriptionProperty(

                subId, COLUMN_PHONE_NUMBER_SOURCE_IMS, "+447539446677");



        // Clean up

        mContextFixture.addCallingOrSelfPermission("");

    }



    @Test

    @SmallTest

    public void testSetPhoneNumberForSourceImsNegativeCases() {

        // In reality the method under test runs in phone process so has MODIFY_PHONE_STATE

        mContextFixture.addCallingOrSelfPermission(MODIFY_PHONE_STATE);

        int subId = 1;

        doReturn(subId).when(mPhone).getSubId();

        SubscriptionInfo subInfo = mock(SubscriptionInfo.class);

        doReturn("gb").when(subInfo).getCountryIso();

        doReturn(subInfo).when(mSubscriptionController).getSubscriptionInfo(subId);



        // 1. No valid phone number; do not set

        Uri[] associatedUris = new Uri[] {

            Uri.parse("sip:447539447777@ims.x.com"),

            Uri.parse("tel:447539446666")

        };

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController, never()).setSubscriptionProperty(

                anyInt(), any(), any());



        // 2. no URI; do not set

        associatedUris = new Uri[] {};

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController, never()).setSubscriptionProperty(

                anyInt(), any(), any());



        // 3. null URI; do not set

        associatedUris = new Uri[] { null };

        mImsPhoneUT.setPhoneNumberForSourceIms(associatedUris);



        verify(mSubscriptionController, never()).setSubscriptionProperty(

                anyInt(), any(), any());



        // 4. null pointer; do not set

        mImsPhoneUT.setPhoneNumberForSourceIms(null);



        verify(mSubscriptionController, never()).setSubscriptionProperty(

                anyInt(), any(), any());



        // Clean up

        mContextFixture.addCallingOrSelfPermission("");

    }



    private ServiceState getServiceStateDataAndVoice(int rat, int regState, boolean isRoaming) {

        ServiceState ss = new ServiceState();

        ss.setStateOutOfService();