> On 11 Dec 2024, at 10:59, Oleg Tselebrovskiy <o.tselebrovs...@postgrespro.ru> 
> wrote:

> When I was looking through static analyzer output I've found a memory leak in 
> pg_dump

Thanks for the report, although 400 bytes won't break the bank in the grand
scheme of things considering how much memory pg_dump use there is little point
in not freeing it.

> Patches for both variants are attached. I also propose changing palloc to 
> pg_malloc
> since it is the only place in pg_dump where palloc is used instead of 
> pg_malloc

Freeing it when not needed anymore seems appropriate, I'll go make that happen.

--
Daniel Gustafsson



Reply via email to