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

Commit cdb2c8f3 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Make MemoryDealer::deallocate private"

parents b935f158 94f0df2d

libs/binder/include/binder/MemoryDealer.h

+2 −1
Original line number Diff line number Diff line
@@ -36,7 +36,6 @@ public: 
            uint32_t flags = 0 /* or bits such as MemoryHeapBase::READ_ONLY */ );



    virtual sp<IMemory> allocate(size_t size);

    virtual void        deallocate(size_t offset);

    virtual void        dump(const char* what) const;



    // allocations are aligned to some value. return that value so clients can account for it.
@@ -48,6 +47,8 @@ protected: 
    virtual ~MemoryDealer();



private:

    friend class Allocation;

    virtual void                deallocate(size_t offset);

    const sp<IMemoryHeap>&      heap() const;

    SimpleBestFitAllocator*     allocator() const;

libs/binder/tests/unit_fuzzers/MemoryDealerFuzz.cpp

+0 −9
Original line number Diff line number Diff line
@@ -45,15 +45,6 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { 
        fdp.PickValueInArray<std::function<void()>>({

                [&]() -> void { dealer->getAllocationAlignment(); },

                [&]() -> void { dealer->getMemoryHeap(); },

                [&]() -> void {

                    size_t offset = fdp.ConsumeIntegral<size_t>();



                    // Offset has already been freed, so return instead.

                    if (free_list.find(offset) != free_list.end()) return;



                    dealer->deallocate(offset);

                    free_list.insert(offset);

                },

                [&]() -> void {

                    std::string randString = fdp.ConsumeRandomLengthString(fdp.remaining_bytes());

                    dealer->dump(randString.c_str());