From 41b87ae313f9fc7ab36fb5fcc070466f33112c25 Mon Sep 17 00:00:00 2001
From: Michael Plass <mplass@google.com>
Date: Fri, 5 Oct 2018 16:38:34 -0700
Subject: [WifiNetworkSelector] use wifiInfo to test sufficiency

Bug: 112196799
Test: Unit tests
Change-Id: I29eb561c299529462775ff81a3e825ae33416846
---
 .../java/com/android/server/wifi/WifiNetworkSelector.java   | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'service')

diff --git a/service/java/com/android/server/wifi/WifiNetworkSelector.java b/service/java/com/android/server/wifi/WifiNetworkSelector.java
index 07fc157d1..4a12b4219 100644
--- a/service/java/com/android/server/wifi/WifiNetworkSelector.java
+++ b/service/java/com/android/server/wifi/WifiNetworkSelector.java
@@ -21,6 +21,7 @@ import android.annotation.Nullable;
 import android.content.Context;
 import android.net.NetworkKey;
 import android.net.wifi.ScanResult;
+import android.net.wifi.SupplicantState;
 import android.net.wifi.WifiConfiguration;
 import android.net.wifi.WifiInfo;
 import android.text.TextUtils;
@@ -81,7 +82,7 @@ public class WifiNetworkSelector {
     /**
      * Interface for WiFi Network Evaluator
      *
-     * A network scorer evaulates all the networks from the scan results and
+     * A network scorer evaluates all the networks from the scan results and
      * recommends the best network in its category to connect or roam to.
      */
     public interface NetworkEvaluator {
@@ -110,7 +111,7 @@ public class WifiNetworkSelector {
          *                       disconnected
          * @param connected      a flag to indicate if ClientModeImpl is in connected
          *                       state
-         * @param untrustedNetworkAllowed a flag to indidate if untrusted networks like
+         * @param untrustedNetworkAllowed a flag to indicate if untrusted networks like
          *                                ephemeral networks are allowed
          * @param connectableNetworks     a list of the ScanDetail and WifiConfiguration
          *                                pair which is used by the WifiLastResortWatchdog
@@ -137,12 +138,12 @@ public class WifiNetworkSelector {
                             mWifiConfigManager.getConfiguredNetwork(wifiInfo.getNetworkId());
 
         // Currently connected?
-        if (network == null) {
+        if (wifiInfo.getSupplicantState() != SupplicantState.COMPLETED) {
             localLog("No current connected network.");
             return false;
         } else {
-            localLog("Current connected network: " + network.SSID
-                    + " , ID: " + network.networkId);
+            localLog("Current connected network: " + wifiInfo.getSSID()
+                    + " , ID: " + wifiInfo.getNetworkId());
         }
 
         int currentRssi = wifiInfo.getRssi();
@@ -156,7 +157,7 @@ public class WifiNetworkSelector {
         }
 
         // Ephemeral network is not qualified.
-        if (network.ephemeral) {
+        if (wifiInfo.isEphemeral()) {
             localLog("Current network is an ephemeral one.");
             return false;
         }
-- 
cgit v1.2.3