commit 322ffcb7240299fd274500db7383a72d5daa3c9d
author Ken Chen <cken@google.com> Mon May 23 22:27:40 2022 +0800
committer Ken Chen <cken@google.com> Mon May 23 22:27:40 2022 +0800
tree 06bb8cf6f4fcf343ab7c46a958cb32324d952699
parent 6235f6490e2a85f823d47b6886fd99f7f884747b

[NetworkStats] Return error on invalid value

Configuration map index 1(CURRENT_STATS_MAP_CONFIGURATION_KEY) can only
have value 0(SELECT_MAP_A) or 1(SELECT_MAP_B). Return error if it is any
other values. Otherwise, read out of array boundary can cause memory
corruption or security issues.

Bug: 231420457
Test: TH
Change-Id: Ia800ad78781f72b8118469c0230cc550796d334e

netd/BpfHandler.cpp

diff --git a/netd/BpfHandler.cpp b/netd/BpfHandler.cpp
index 08e66d9..d6f2949 100644
--- a/netd/BpfHandler.cpp
+++ b/netd/BpfHandler.cpp
@@ -104,6 +104,7 @@
     RETURN_IF_NOT_OK(mConfigurationMap.writeValue(CURRENT_STATS_MAP_CONFIGURATION_KEY, SELECT_MAP_A,
                                                   BPF_ANY));
     RETURN_IF_NOT_OK(mUidPermissionMap.init(UID_PERMISSION_MAP_PATH));
+    ALOGI("%s successfully", __func__);
 
     return netdutils::status::ok;
 }