diff options
author | Quang Anh Luong <qal@google.com> | 2023-11-21 16:54:07 +0900 |
---|---|---|
committer | Quang Anh Luong <qal@google.com> | 2023-11-21 16:58:34 +0900 |
commit | 71f65e5393c4cbe2cb42374a51a06276ec85ec87 (patch) | |
tree | 66e19f2e03b980867a6896303cac9ee2749659b8 | |
parent | cc5dcf59073c57fedb4fcb3e7d9f4a531c751cdd (diff) | |
download | wifi-71f65e5393c4cbe2cb42374a51a06276ec85ec87.tar.gz |
Fix NPE when shared connectivity service connected
Fix NPE caused by hotspot networks/known networks being null when the
shared connectivity service is connected.
Bug: 311139310
Test: atest WifiTrackerLibTests
Change-Id: I52fe192f80572bad5de466af4903655573a566fa
-rw-r--r-- | libs/WifiTrackerLib/src/com/android/wifitrackerlib/WifiPickerTracker.java | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/libs/WifiTrackerLib/src/com/android/wifitrackerlib/WifiPickerTracker.java b/libs/WifiTrackerLib/src/com/android/wifitrackerlib/WifiPickerTracker.java index 6401f094f..72bbf0f55 100644 --- a/libs/WifiTrackerLib/src/com/android/wifitrackerlib/WifiPickerTracker.java +++ b/libs/WifiTrackerLib/src/com/android/wifitrackerlib/WifiPickerTracker.java @@ -507,9 +507,15 @@ public class WifiPickerTracker extends BaseWifiTracker { protected void handleServiceConnected() { if (mInjector.isSharedConnectivityFeatureEnabled()) { mKnownNetworkDataCache.clear(); - mKnownNetworkDataCache.addAll(mSharedConnectivityManager.getKnownNetworks()); + List<KnownNetwork> knownNetworks = mSharedConnectivityManager.getKnownNetworks(); + if (knownNetworks != null) { + mKnownNetworkDataCache.addAll(knownNetworks); + } mHotspotNetworkDataCache.clear(); - mHotspotNetworkDataCache.addAll(mSharedConnectivityManager.getHotspotNetworks()); + List<HotspotNetwork> hotspotNetworks = mSharedConnectivityManager.getHotspotNetworks(); + if (hotspotNetworks != null) { + mHotspotNetworkDataCache.addAll(hotspotNetworks); + } updateKnownNetworkEntryScans(mScanResultUpdater.getScanResults()); updateHotspotNetworkEntries(); updateWifiEntries(); |