commit 79365da15181db2e7d9622db834c4c0b84f1e060
author Maciej Żenczykowski <maze@google.com> Fri Jan 17 19:07:37 2020 -0800
committer Maciej Żenczykowski <maze@google.com> Sat Jan 18 19:32:09 2020 -0800
tree 39bee2b2156d139cc4c2daf7a7b6ee629c883621
parent b479fd6a9f2b96099e86fd1a47002de904340c4f

BpfMap/Utils - fix const-ness of key & value

When accessing maps keys are always const/read-only.
Similarly for the value when it is being stored in the map.

This eliminates four const_casts.

Test: build, atest
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Icc57b46a8e9e81b5b06264260bfbf34a4ba9105d

libbpf_android/include/bpf/BpfMap.h

diff --git a/libbpf_android/include/bpf/BpfMap.h b/libbpf_android/include/bpf/BpfMap.h
index a894b57..80ae62b 100644
--- a/libbpf_android/include/bpf/BpfMap.h
+++ b/libbpf_android/include/bpf/BpfMap.h
@@ -79,7 +79,7 @@
 
     netdutils::StatusOr<Key> getNextKey(const Key& key) const {
         Key nextKey;
-        if (getNextMapKey(mMapFd, const_cast<Key*>(&key), &nextKey)) {
+        if (getNextMapKey(mMapFd, &key, &nextKey)) {
             return netdutils::statusFromErrno(
                 errno, base::StringPrintf("Get next key of map %d failed", mMapFd.get()));
         }
@@ -87,7 +87,7 @@
     }
 
     netdutils::Status writeValue(const Key& key, const Value& value, uint64_t flags) {
-        if (writeToMapEntry(mMapFd, const_cast<Key*>(&key), const_cast<Value*>(&value), flags)) {
+        if (writeToMapEntry(mMapFd, &key, &value, flags)) {
             return netdutils::statusFromErrno(
                 errno, base::StringPrintf("write to map %d failed", mMapFd.get()));
         }
@@ -96,7 +96,7 @@
 
     netdutils::StatusOr<Value> readValue(const Key key) const {
         Value value;
-        if (findMapEntry(mMapFd, const_cast<Key*>(&key), &value)) {
+        if (findMapEntry(mMapFd, &key, &value)) {
             return netdutils::statusFromErrno(
                 errno, base::StringPrintf("read value of map %d failed", mMapFd.get()));
         }
@@ -104,7 +104,7 @@
     }
 
     netdutils::Status deleteValue(const Key& key) {
-        if (deleteMapEntry(mMapFd, const_cast<Key*>(&key))) {
+        if (deleteMapEntry(mMapFd, &key)) {
             return netdutils::statusFromErrno(
                 errno, base::StringPrintf("delete entry from map %d failed", mMapFd.get()));
         }