On Mon, Mar 09, 2026 at 11:21:35AM +0800, yangyz wrote: > I think it should be modified. > > Move createPQExpBuffer inside the conditional block to match its destroy > counterpart. > This improves code clarity and satisfies static analyzers, even though the > actual memory > leak is minimal in practice.
destroyPQExpBuffer() is called for each tuple from pg_database except
if dealing with "template{0,1}" or "postgres". It means that we would
just leak a few bytes for these three cases. I agree that the
variable declaration can be placed better, but it's really not worth
bothering in this context.
--
Michael
signature.asc
Description: PGP signature
