Re: [PATCH] Fix memory leak in pg_config - Mailing list pgsql-hackers

From Andres Freund
Subject Re: [PATCH] Fix memory leak in pg_config
Date
Msg-id t7osyzcabasm2rz6abp2v4wcjxkn2ogm7xbrsyhle2suirlmoy@uf5zxspsb77i
Whole thread
In response to Re: [PATCH] Fix memory leak in pg_config  (Ivan Kush <ivan.kush@tantorlabs.com>)
List pgsql-hackers
Hi,

On 2026-07-22 14:54:17 +0300, Ivan Kush wrote:
> If the project policy is to ignore leaks in short-lived binaries, why
> are those known cases not listed explicitly in an LSan suppression file (for
> example, see attached patch)?
> 
> Disabling leak detection globally with detect_leaks=0 also hides leaks
> that may be relevant in long-running processes.  Explicit suppressions
> would document the intentionally ignored cases while preserving LSan
> coverage for the rest of the code.

Because we so far have not been using lsan. It might be worth supporting it,
but we haven't so far. You'd have to make a case for doing that on its own.

For the server lsan is probably not very helpful. Just about all allocations
in the backend are done via memory contexts, which lsan does not understand -
and can't easily be made to be understand, IIRC. Whereas we made valgrind
mostly understand memory contexts.

Greetings,

Andres Freund



Attachment

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: incremental backup issue
Next
From: Andres Freund
Date:
Subject: Re: convert various variables to atomics