commit 21ce721bc087d679686acedd06ec7d2b19cad164
author Maciej Żenczykowski <maze@google.com> Mon Jun 13 17:28:41 2022 -0700
committer Maciej Żenczykowski <maze@google.com> Mon Jun 13 19:39:06 2022 -0700
tree 92f20fd323302e1469d051e6916a1da1272bc047
parent ffffa35ee8510dee3fc44e58b834986bf82bf302

BpfHandler.h - switch mStatsMapB from BpfMap to BpfMapRO

We'd like to switch more of these, but this is the only one
where tests don't get in our way.

We'll fix the rest later.  This gives us an *example* for
further testing.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I9b280c7ba2d19644fc3cdd0c132de85f45df69b4

netd/BpfHandler.cpp

diff --git a/netd/BpfHandler.cpp b/netd/BpfHandler.cpp
index 42d0de5..5ae8ab6 100644
--- a/netd/BpfHandler.cpp
+++ b/netd/BpfHandler.cpp
@@ -199,6 +199,7 @@
 
     BpfMap<StatsKey, StatsValue>& currentMap =
             (configuration.value() == SELECT_MAP_A) ? mStatsMapA : mStatsMapB;
+    // HACK: mStatsMapB becomes RW BpfMap here, but countUidStatsEntries doesn't modify so it works
     base::Result<void> res = currentMap.iterate(countUidStatsEntries);
     if (!res.ok()) {
         ALOGE("Failed to count the stats entry in map %d: %s", currentMap.getMap().get(),