Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 92f083d3 authored by Maciej Żenczykowski's avatar Maciej Żenczykowski
Browse files

GpuMemTracerTest - 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: default avatarMaciej Żenczykowski <maze@google.com>
Change-Id: I4a8a3b918f25501ab0ab4aa74ce5d2a629ece463
parent 099a4648

services/gpuservice/tests/unittests/GpuMemTracerTest.cpp

+3 −3
Original line number Diff line number Diff line
@@ -64,11 +64,11 @@ public: 
        mTestableGpuMem = TestableGpuMem(mGpuMem.get());



        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());

    }