commit 466a722208a38f2066a5a699a1f4a7fe529873f3
author Maciej Żenczykowski <maze@google.com> Sat Dec 10 12:16:58 2022 +0000
committer Cherrypicker Worker <android-build-cherrypicker-worker@google.com> Mon Dec 26 10:03:26 2022 +0000
tree c5ad2cd52b9f53e08da8d6ce4e052021eaf2d2fd
parent 00791401fe00064618f6b9c40dc30b9e7b8caabf

remove only use of OVERFLOW_COUNTERSET

See:
  https://source.corp.google.com/search?q=p:android$%20OVERFLOW_COUNTERSET

(I'm guessing any remaining uses are in Java not C/C++?)

I'd like to remove the definition from the
  frameworks/libs/net/common/native/bpf_headers/include/bpf/BpfUtils.h
header file, where it really doesn't fit.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I687f4ca0f52c362b29be7fe612f90d8aed2afe9e
(cherry picked from commit 9796860b2562ed995e5dcb56621aa299043d204c)
Merged-In: I687f4ca0f52c362b29be7fe612f90d8aed2afe9e

service-t/native/libs/libnetworkstats/BpfNetworkStatsTest.cpp

diff --git a/service-t/native/libs/libnetworkstats/BpfNetworkStatsTest.cpp b/service-t/native/libs/libnetworkstats/BpfNetworkStatsTest.cpp
index 6f9c8c2..ff62c0b 100644
--- a/service-t/native/libs/libnetworkstats/BpfNetworkStatsTest.cpp
+++ b/service-t/native/libs/libnetworkstats/BpfNetworkStatsTest.cpp
@@ -326,7 +326,8 @@
             .txPackets = TEST_PACKET1,
             .txBytes = TEST_BYTES1,
     };
-    populateFakeStats(0, 0, 0, OVERFLOW_COUNTERSET, value1, mFakeStatsMap);
+    // next stats entry will be ignored due to ifindex 0 not being present in mFakeIfaceIndexNameMap
+    populateFakeStats(0, 0, 0, TEST_COUNTERSET1, value1, mFakeStatsMap);
     populateFakeStats(TEST_UID1, 0, IFACE_INDEX1, TEST_COUNTERSET0, value1, mFakeStatsMap);
     populateFakeStats(TEST_UID1, 0, IFACE_INDEX2, TEST_COUNTERSET0, value1, mFakeStatsMap);
     populateFakeStats(TEST_UID1, 0, IFACE_INDEX1, TEST_COUNTERSET1, value1, mFakeStatsMap);