Re: [PATCH] Fix memory leak in pg_config

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