Memory leak fix in rmtree.c

From: Ильясов Ян <ianilyasov(at)outlook(dot)com>
To: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Memory leak fix in rmtree.c
Date: 2024-02-06 09:34:42
Message-ID: GV1P251MB1004EB38F3330E50FA64F24ACD462@GV1P251MB1004.EURP251.PROD.OUTLOOK.COM
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello hackers,

Just like some of my colleagues I've been using Svace*
and I think I've found a bug in src/common/rmtree.c .

In 64th line function returns false in case it couldn't open a directory,
but the memory, that have been allocated for char** dirnames is
not freed.

The patch that has a fix of this is attached and is based on the latest
master code.

* ​- https://svace.pages.ispras.ru/svace-website/en/

Kind regards,
Ian Ilyasov.

Attachment Content-Type Size
v001-Fix_memory_leak_in_rmtree.patch text/x-patch 607 bytes

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2024-02-06 09:40:27 Re: Memory leak fix in rmtree.c
Previous Message Ashutosh Bapat 2024-02-06 09:26:31 Re: pgsql: Add EXPLAIN (MEMORY) to report planner memory consumption