commit d2d09358c143e435406e9f49f3ec9cdcef352093
author Maciej Żenczykowski <maze@google.com> Thu Jun 16 19:08:28 2022 +0000
committer Maciej Żenczykowski <maze@google.com> Thu Jun 16 23:59:57 2022 +0000
tree c2538b419ea3e8b2e4ed9711e13f534d52464bb2
parent 099a4648fdf4f261f70ca63920d24ba963d652f8

GpuMemTest - switch mTestMap bpfmap init to move assignment

also remove use of .getMap() which I want to remove as it exposes fds
and can leak (isValid check in the next line is equivalent)

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

services/gpuservice/tests/unittests/GpuMemTest.cpp

diff --git a/services/gpuservice/tests/unittests/GpuMemTest.cpp b/services/gpuservice/tests/unittests/GpuMemTest.cpp
index e916221..2c19c24 100644
--- a/services/gpuservice/tests/unittests/GpuMemTest.cpp
+++ b/services/gpuservice/tests/unittests/GpuMemTest.cpp
@@ -65,11 +65,11 @@
         mTestableGpuMem = TestableGpuMem(mGpuMem.get());
         mTestableGpuMem.setInitialized();
         errno = 0;
-        mTestMap = bpf::BpfMap<uint64_t, uint64_t>(BPF_MAP_TYPE_HASH, TEST_MAP_SIZE,
-                                                   BPF_F_NO_PREALLOC);
+        mTestMap = std::move(bpf::BpfMap<uint64_t, uint64_t>(BPF_MAP_TYPE_HASH,
+                                                             TEST_MAP_SIZE,
+                                                             BPF_F_NO_PREALLOC));
 
         EXPECT_EQ(0, errno);
-        EXPECT_LE(0, mTestMap.getMap().get());
         EXPECT_TRUE(mTestMap.isValid());
     }