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

Commit 9a00218d authored by Jack Yu's avatar Jack Yu
Browse files

Prevent unnecessary building APN candidates 

When the APN context is already in disconnected state, there
is no need to build the waiting APN list, which can potentially
cause preferred APN gets deleted.

Bug: 144400654
Test: Manual
Change-Id: I8dd8edc6a9485148dde7b9b6b4326baed1632ea4
parent fa869f4f

src/java/com/android/internal/telephony/dataconnection/DcTracker.java

+17 −21
Original line number Diff line number Diff line
@@ -3322,17 +3322,12 @@ public class DcTracker extends Handler { 
                    apnList = sortApnListByPreferred(apnList);

                    if (DBG) log("buildWaitingApns: X added preferred apnList=" + apnList);

                    return apnList;

                } else {

                    if (DBG) log("buildWaitingApns: no preferred APN");

                    setPreferredApn(-1);

                    mPreferredApn = null;

                }

            } else {

            }

            if (DBG) log("buildWaitingApns: no preferred APN");

            setPreferredApn(-1);

            mPreferredApn = null;

        }

        }



        if (DBG) log("buildWaitingApns: mAllApnSettings=" + mAllApnSettings);

        for (ApnSetting apn : mAllApnSettings) {
@@ -4312,6 +4307,7 @@ public class DcTracker extends Handler { 
                return;

            }

            for (ApnContext apnContext : mApnContexts.values()) {

                if (!apnContext.isDisconnected()) {

                    ArrayList<ApnSetting> currentWaitingApns = apnContext.getWaitingApns();

                    ArrayList<ApnSetting> waitingApns = buildWaitingApns(

                            apnContext.getApnType(), getDataRat());
@@ -4320,11 +4316,11 @@ public class DcTracker extends Handler { 
                            && ((waitingApns.size() != currentWaitingApns.size())

                            // Check if the existing waiting APN list can cover the newly built APN

                            // list. If yes, then we don't need to tear down the existing data call.

                        // TODO: We probably need to rebuild APN list when roaming status changes.

                            // TODO: We probably need to rebuild APN list when roaming status

                            //  changes.

                            || !containsAllApns(currentWaitingApns, waitingApns))) {

                        if (VDBG) log("new waiting apn is different for " + apnContext);

                        apnContext.setWaitingApns(waitingApns);

                    if (!apnContext.isDisconnected()) {

                        if (VDBG) log("cleanUpConnectionsOnUpdatedApns for " + apnContext);

                        apnContext.setReason(reason);

                        cleanUpConnectionInternal(true, RELEASE_TYPE_DETACH, apnContext);