On Fri, Sep 12, 2025 at 1:54 PM Ranier Vilela <ranier...@gmail.com> wrote: > Per Coverity. > > Coverity reports this resource leak in test_binaryheap module. > I think that is right. > > Trivial patch attached.
If this were correct, we'd need to also free the memory in all the error paths. But of course, in both error and non-error paths, we rely on memory context cleanup to free memory for us, except in cases where there's some specific reason to believe that's not good enough. I doubt that there is any such reason in this case. See src/backend/utils/mmgr/README -- Robert Haas EDB: http://www.enterprisedb.com
