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

Commit a867306b authored by Jack Yu's avatar Jack Yu
Browse files

Fixed cell broadcast configuration not sent when no sim is inserted 

If no SIM is inserted, use the default phone instead of failing
the request.

Test: Manual
bug: 63903157
Merged-In: I9841904775f1422b1ff3d02d520eaf29f1393ffd
Change-Id: I9841904775f1422b1ff3d02d520eaf29f1393ffd
parent 7d64fae5

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

+4 −8
Original line number Diff line number Diff line
@@ -16,9 +16,6 @@ 


package com.android.internal.telephony;



import java.util.ArrayList;

import java.util.Random;



import android.content.Context;

import android.content.Intent;

import android.os.AsyncResult;
@@ -31,13 +28,12 @@ import android.telephony.Rlog; 
import android.telephony.TelephonyManager;

import android.util.Log;



import com.android.internal.telephony.PhoneSwitcher;

import com.android.internal.telephony.uicc.UiccController;



import java.io.FileDescriptor;

import java.io.PrintWriter;

import java.util.concurrent.atomic.AtomicInteger;

import java.util.ArrayList;

import java.util.HashSet;

import java.util.Random;

import java.util.concurrent.atomic.AtomicInteger;



public class ProxyController {

    static final String LOG_TAG = "ProxyController";
@@ -130,7 +126,7 @@ public class ProxyController { 


        mUiccPhoneBookController = new UiccPhoneBookController(mPhones);

        mPhoneSubInfoController = new PhoneSubInfoController(mContext, mPhones);

        mUiccSmsController = new UiccSmsController(mPhones);

        mUiccSmsController = new UiccSmsController();

        mSetRadioAccessFamilyStatus = new int[mPhones.length];

        mNewRadioAccessFamily = new int[mPhones.length];

        mOldRadioAccessFamily = new int[mPhones.length];

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

+19 −41
Original line number Diff line number Diff line
@@ -42,16 +42,20 @@ import java.util.List; 
public class UiccSmsController extends ISms.Stub {

    static final String LOG_TAG = "RIL_UiccSmsController";



    protected Phone[] mPhone;



    protected UiccSmsController(Phone[] phone){

        mPhone = phone;



    protected UiccSmsController() {

        if (ServiceManager.getService("isms") == null) {

            ServiceManager.addService("isms", this);

        }

    }



    private Phone getPhone(int subId) {

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

        if (phone == null) {

            phone = PhoneFactory.getDefaultPhone();

        }

        return phone;

    }



    @Override

    public boolean

    updateMessageOnIccEfForSubscriber(int subId, String callingPackage, int index, int status,
@@ -320,45 +324,26 @@ public class UiccSmsController extends ISms.Stub { 
    }



    /**

     * get sms interface manager object based on subscription.

     **/

     * Get sms interface manager object based on subscription.

     * @return ICC SMS manager

     */

    private @Nullable IccSmsInterfaceManager getIccSmsInterfaceManager(int subId) {

        if (!isActiveSubId(subId)) {

            Rlog.e(LOG_TAG, "Subscription " + subId + " is inactive.");

            return null;

        }



        int phoneId = SubscriptionController.getInstance().getPhoneId(subId) ;

        //Fixme: for multi-subscription case

        if (!SubscriptionManager.isValidPhoneId(phoneId)

                || phoneId == SubscriptionManager.DEFAULT_PHONE_INDEX) {

            phoneId = 0;

        }



        try {

            return (IccSmsInterfaceManager)

                ((Phone)mPhone[(int)phoneId]).getIccSmsInterfaceManager();

        } catch (NullPointerException e) {

            Rlog.e(LOG_TAG, "Exception is :"+e.toString()+" For subscription :"+subId );

            e.printStackTrace();

            return null;

        } catch (ArrayIndexOutOfBoundsException e) {

            Rlog.e(LOG_TAG, "Exception is :"+e.toString()+" For subscription :"+subId );

            e.printStackTrace();

            return null;

        }

        return getPhone(subId).getIccSmsInterfaceManager();

    }



    /**

       Gets User preferred SMS subscription */

     * Get User preferred SMS subscription

     * @return User preferred SMS subscription

     */

    @Override

    public int getPreferredSmsSubscription() {

        return SubscriptionController.getInstance().getDefaultSmsSubId();

    }



    /**

     * Get SMS prompt property,  enabled or not

     **/

     * Get SMS prompt property enabled or not

     * @return True if SMS prompt is enabled.

     */

    @Override

    public boolean isSMSPromptEnabled() {

        return PhoneFactory.isSMSPromptEnabled();
@@ -392,13 +377,6 @@ public class UiccSmsController extends ISms.Stub { 
        }

    }



    /*

     * @return true if the subId is active.

     */

    private boolean isActiveSubId(int subId) {

        return SubscriptionController.getInstance().isActiveSubId(subId);

    }



    private void sendErrorInPendingIntent(@Nullable PendingIntent intent, int errorCode) {

        if (intent != null) {

            try {