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

Commit 2236bf11 authored by Wink Saville's avatar Wink Saville
Browse files

getActiveSubscriptionInfoList used wrong slection operator. 

The call to getSubInfo should use ">=0" to find "active" subscriptions.
Previously INVALID_SIM_SLOT_INDEX was -1000 and now its -1 so the
"!=" would return older inactive subscriptons (i.e. -1000 for
SIM_SLOT_INDEX) as well as the truly active subscriptions.

Bug: 18599334
Change-Id: Ied1af25773df34725508a2ab1a5954e6a3ffb20a
parent d2342663

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

+1 −2
Original line number Diff line number Diff line
@@ -494,8 +494,7 @@ public class SubscriptionController extends ISub.Stub { 
            return subList;

        }



        subList = getSubInfo(SubscriptionManager.SIM_SLOT_INDEX

                + "!=" + SubscriptionManager.INVALID_SIM_SLOT_INDEX, null);

        subList = getSubInfo(SubscriptionManager.SIM_SLOT_INDEX + ">=0", null);

        if (subList != null) {

            // FIXME: Unnecessary when an insertion sort is used!

            Collections.sort(subList, new Comparator<SubscriptionInfo>() {