From 330a7ccd895b4047323d4b633e36489daf1f8f4c Mon Sep 17 00:00:00 2001
From: Michael Plass <mplass@google.com>
Date: Fri, 17 Jan 2020 16:56:49 -0800
Subject: Remove WifiScoreCard entries when forgetting a network

Bug: 146460486
Test: atest FrameworksWifiTests FrameworksWifiApiTests
Test: Manual - Check that the entries for wifi networks are removed
      as those networks are forgotten.
      adb pull /data/user_de/0/com.android.networkstack/databases/IpMemoryStore.db
      sqlite3 IpMemoryStore.db .dump
Change-Id: I2fee17aa4cfcd2afbd3e7d36937e2bf8ae4f3313
---
 .../com/android/server/wifi/MemoryStoreImpl.java   | 30 ++++++++++++++++++++++
 .../com/android/server/wifi/WifiScoreCard.java     | 26 ++++++++++++++-----
 2 files changed, 49 insertions(+), 7 deletions(-)

(limited to 'service')

diff --git a/service/java/com/android/server/wifi/MemoryStoreImpl.java b/service/java/com/android/server/wifi/MemoryStoreImpl.java
index 1b819d46d..a15ae8106 100644
--- a/service/java/com/android/server/wifi/MemoryStoreImpl.java
+++ b/service/java/com/android/server/wifi/MemoryStoreImpl.java
@@ -21,6 +21,7 @@ import android.annotation.Nullable;
 import android.content.Context;
 import android.net.IpMemoryStore;
 import android.net.ipmemorystore.Blob;
+import android.net.ipmemorystore.NetworkAttributes;
 import android.net.ipmemorystore.Status;
 import android.util.Log;
 
@@ -127,6 +128,35 @@ final class MemoryStoreImpl implements WifiScoreCard.MemoryStore {
         }
     }
 
+    @Override
+    public void setCluster(String key, String cluster) {
+        if (mBroken) return;
+        try {
+            NetworkAttributes attributes = new NetworkAttributes.Builder()
+                    .setCluster(cluster)
+                    .build();
+            mIpMemoryStore.storeNetworkAttributes(key, attributes, status -> {
+                Log.d(TAG, "Set cluster " + cluster + " for " + key + ": " + status);
+            });
+        } catch (RuntimeException e) {
+            handleException(e);
+        }
+    }
+
+    @Override
+    public void removeCluster(String cluster) {
+        if (mBroken) return;
+        try {
+            final boolean needWipe = true;
+            mIpMemoryStore.deleteCluster(cluster, needWipe, (status, deletedRecords) -> {
+                Log.d(TAG, "Remove cluster " + cluster + ": " + status
+                        + " deleted: " + deletedRecords);
+            });
+        } catch (RuntimeException e) {
+            handleException(e);
+        }
+    }
+
     /**
      * Starts using IpMemoryStore.
      */
diff --git a/service/java/com/android/server/wifi/WifiScoreCard.java b/service/java/com/android/server/wifi/WifiScoreCard.java
index 346a8febe..ca77e8396 100644
--- a/service/java/com/android/server/wifi/WifiScoreCard.java
+++ b/service/java/com/android/server/wifi/WifiScoreCard.java
@@ -136,6 +136,10 @@ public class WifiScoreCard {
         void read(String key, String name, BlobListener blobListener);
         /** Requests a write, does not wait for completion */
         void write(String key, String name, byte[] value);
+        /** Sets the cluster identifier */
+        void setCluster(String key, String cluster);
+        /** Requests removal of all entries matching the cluster */
+        void removeCluster(String cluster);
     }
     /** Asynchronous response to a read request */
     public interface BlobListener {
@@ -249,9 +253,15 @@ public class WifiScoreCard {
         if (perBssid == mDummyPerBssid) {
             return new Pair<>(null, null);
         }
-        final long groupIdHash = computeHashLong(
-                perBssid.ssid, mDummyPerBssid.bssid, mL2KeySeed);
-        return new Pair<>(perBssid.getL2Key(), groupHintFromLong(groupIdHash));
+        return new Pair<>(perBssid.getL2Key(), groupHintFromSsid(perBssid.ssid));
+    }
+
+    /**
+     * Computes the GroupHint associated with the given ssid.
+     */
+    public @NonNull String groupHintFromSsid(String ssid) {
+        final long groupIdHash = computeHashLong(ssid, mDummyPerBssid.bssid, mL2KeySeed);
+        return groupHintFromLong(groupIdHash);
     }
 
     /**
@@ -1447,6 +1457,7 @@ public class WifiScoreCard {
                 Log.e(TAG, "More answers than we expected!");
             }
         }
+
         /**
          * Handles (when convenient) the arrival of previously stored data.
          *
@@ -1459,7 +1470,6 @@ public class WifiScoreCard {
             return mPendingReadFromStore.getAndSet(null);
         }
 
-
         int idFromLong() {
             return (int) mHash & 0x7fffffff;
         }
@@ -1552,10 +1562,9 @@ public class WifiScoreCard {
             return;
         }
         mApForNetwork.remove(ssid);
+        mApForBssid.entrySet().removeIf(entry -> ssid.equals(entry.getValue().ssid));
         if (mMemoryStore == null) return;
-        PerNetwork ans = new PerNetwork(ssid);
-        byte[] serialized = {};
-        mMemoryStore.write(ans.getL2Key(), PER_NETWORK_DATA_NAME, serialized);
+        mMemoryStore.removeCluster(groupHintFromSsid(ssid));
     }
 
     void requestReadNetwork(final PerNetwork perNetwork) {
@@ -1586,7 +1595,9 @@ public class WifiScoreCard {
             if (perBssid.changed) {
                 perBssid.finishPendingRead();
                 byte[] serialized = perBssid.toAccessPoint(/* No BSSID */ true).toByteArray();
+                mMemoryStore.setCluster(perBssid.getL2Key(), groupHintFromSsid(perBssid.ssid));
                 mMemoryStore.write(perBssid.getL2Key(), PER_BSSID_DATA_NAME, serialized);
+
                 perBssid.changed = false;
                 count++;
                 bytes += serialized.length;
@@ -1606,6 +1617,7 @@ public class WifiScoreCard {
             if (perNetwork.changed) {
                 perNetwork.finishPendingRead();
                 byte[] serialized = perNetwork.toNetworkStats().toByteArray();
+                mMemoryStore.setCluster(perNetwork.getL2Key(), groupHintFromSsid(perNetwork.ssid));
                 mMemoryStore.write(perNetwork.getL2Key(), PER_NETWORK_DATA_NAME, serialized);
                 perNetwork.changed = false;
                 count++;
-- 
cgit v1.2.3