Merge "validation success - remove BSSID from blocklist"

1 files changed, 15 insertions, 0 deletions

diff --git a/service/java/com/android/server/wifi/BssidBlocklistMonitor.java b/service/java/com/android/server/wifi/BssidBlocklistMonitor.java
index 2151da2af..530ad6de0 100644
--- a/service/java/com/android/server/wifi/BssidBlocklistMonitor.java
+++ b/service/java/com/android/server/wifi/BssidBlocklistMonitor.java
@@ -279,6 +279,7 @@ public class BssidBlocklistMonitor {
 
     /**
      * Note a successful network validation on a BSSID and clear appropriate failure counters.
+     * And then remove the BSSID from blocklist.
      */
     public void handleNetworkValidationSuccess(@NonNull String bssid, @NonNull String ssid) {
         mWifiScoreCard.resetBssidBlocklistStreak(ssid, bssid, REASON_NETWORK_VALIDATION_FAILURE);
@@ -287,6 +288,12 @@ public class BssidBlocklistMonitor {
             return;
         }
         status.failureCount[REASON_NETWORK_VALIDATION_FAILURE] = 0;
+        /**
+         * Network validation may take more than 1 tries to succeed.
+         * remove the BSSID from blocklist to make sure we are not accidentally blocking good
+         * BSSIDs.
+         **/
+        status.removeFromBlocklist();
     }
 
     /**
@@ -441,6 +448,14 @@ public class BssidBlocklistMonitor {
             blocklistEndTimeMs = mClock.getWallClockMillis() + durationMs;
         }
 
+        /**
+         * Remove this BSSID from the blocklist.
+         */
+        public void removeFromBlocklist() {
+            isInBlocklist = false;
+            blocklistEndTimeMs = 0;
+        }
+
         @Override
         public String toString() {
             StringBuilder sb = new StringBuilder();