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

Commit e160c0ca authored by Kai Shi's avatar Kai Shi
Browse files

Bug fix: LCE list initialization 

Initialize LCE list so that its size is at least 1. This is important for 1st call back after registration especially when there is no LCE report yet.

Bug: 198682652
Test: atest -c TelephonyCallbackTest in airplane mode.
Change-Id: I54b4ce0c51e6c10c6ef67776cf3083a04b7d5da3
parent e88edfdd

services/core/java/com/android/server/TelephonyRegistry.java

+6 −2
Original line number Diff line number Diff line
@@ -385,6 +385,10 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { 
    private int[] mAllowedNetworkTypeReason;

    private long[] mAllowedNetworkTypeValue;



    private static final List<LinkCapacityEstimate> INVALID_LCE_LIST =

            new ArrayList<LinkCapacityEstimate>(Arrays.asList(new LinkCapacityEstimate(

            LinkCapacityEstimate.LCE_TYPE_COMBINED,

            LinkCapacityEstimate.INVALID, LinkCapacityEstimate.INVALID)));

    private List<List<LinkCapacityEstimate>> mLinkCapacityEstimateLists;



    /**
@@ -719,7 +723,7 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { 
            mPhysicalChannelConfigs.add(i, new ArrayList<>());

            mAllowedNetworkTypeReason[i] = -1;

            mAllowedNetworkTypeValue[i] = -1;

            mLinkCapacityEstimateLists.add(i, new ArrayList<>());

            mLinkCapacityEstimateLists.add(i, INVALID_LCE_LIST);

        }

    }
@@ -819,7 +823,7 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub { 
            mPhysicalChannelConfigs.add(i, new ArrayList<>());

            mAllowedNetworkTypeReason[i] = -1;

            mAllowedNetworkTypeValue[i] = -1;

            mLinkCapacityEstimateLists.add(i, new ArrayList<>());

            mLinkCapacityEstimateLists.add(i, INVALID_LCE_LIST);

        }



        mAppOps = mContext.getSystemService(AppOpsManager.class);