diff options
author | Nagendra Prasad Nagarle Basavaraju <nagendranb@google.com> | 2023-04-05 09:18:43 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-04-05 09:18:43 +0000 |
commit | 4bd819e2bd32e66c8922a5d5ad7779d2a0127f2e (patch) | |
tree | 7a27759c0e4aa4bfdab835dfebe77a99db8b6de6 | |
parent | fb16afbf0be481d29c127d918b164339dca88d24 (diff) | |
parent | 0ec6955ec5a701034c1624e2ebf6be1aa08238a1 (diff) | |
download | Telephony-4bd819e2bd32e66c8922a5d5ad7779d2a0127f2e.tar.gz |
[HysteresisDB] Fix NullPointerException at CellularQualityMonitor am: 0ec6955ec5
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/Telephony/+/22410468
Change-Id: I56509cbdd21b97f5077a2d40f17924b474584571
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r-- | services/QualifiedNetworksService/src/com/android/telephony/qns/QnsComponents.java | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/services/QualifiedNetworksService/src/com/android/telephony/qns/QnsComponents.java b/services/QualifiedNetworksService/src/com/android/telephony/qns/QnsComponents.java index 5862bc2..0471ea8 100644 --- a/services/QualifiedNetworksService/src/com/android/telephony/qns/QnsComponents.java +++ b/services/QualifiedNetworksService/src/com/android/telephony/qns/QnsComponents.java @@ -72,12 +72,6 @@ class QnsComponents { mCellularNetworkStatusTrackers.put( slotId, new CellularNetworkStatusTracker(mQnsTelephonyListeners.get(slotId), slotId)); - mCellularQualityMonitors.put( - slotId, - new CellularQualityMonitor(mContext, - mQnsCarrierConfigManagers.get(slotId), - mQnsTelephonyListeners.get(slotId), - slotId)); mQnsProvisioningListeners.put( slotId, new QnsProvisioningListener(mContext, mQnsImsManagers.get(slotId), slotId)); mQnsEventDispatchers.put( @@ -90,6 +84,12 @@ class QnsComponents { mQnsCarrierConfigManagers.put( slotId, new QnsCarrierConfigManager(mContext, mQnsEventDispatchers.get(slotId), slotId)); + mCellularQualityMonitors.put( + slotId, + new CellularQualityMonitor(mContext, + mQnsCarrierConfigManagers.get(slotId), + mQnsTelephonyListeners.get(slotId), + slotId)); if (mQnsTimer == null) { mQnsTimer = new QnsTimer(mContext); } @@ -266,6 +266,11 @@ class QnsComponents { mQnsTimer.close(); mQnsTimer = null; } + CellularQualityMonitor cellularQualityMonitor = mCellularQualityMonitors.get(slotId); + if (cellularQualityMonitor != null) { + mCellularQualityMonitors.remove(slotId); + cellularQualityMonitor.close(); + } QnsCarrierConfigManager qnsCarrierConfigManager = mQnsCarrierConfigManagers.get(slotId); if (qnsCarrierConfigManager != null) { mQnsCarrierConfigManagers.remove(slotId); @@ -281,11 +286,6 @@ class QnsComponents { mQnsProvisioningListeners.remove(slotId); qnsProvisioningListener.close(); } - CellularQualityMonitor cellularQualityMonitor = mCellularQualityMonitors.get(slotId); - if (cellularQualityMonitor != null) { - mCellularQualityMonitors.remove(slotId); - cellularQualityMonitor.close(); - } CellularNetworkStatusTracker cellularTracker = mCellularNetworkStatusTrackers.get(slotId); if (cellularTracker != null) { mCellularNetworkStatusTrackers.remove(slotId); |