RE: Memory leak in pg_logical_slot_{get,peek}_changes - Mailing list pgsql-hackers

From Zhijie Hou (Fujitsu)
Subject RE: Memory leak in pg_logical_slot_{get,peek}_changes
Date
Msg-id OS0PR01MB57160B4CAF3BB995155705C8940E2@OS0PR01MB5716.jpnprd01.prod.outlook.com
Whole thread Raw
List pgsql-hackers
On Thursday, December 26, 2024 8:01 PM vignesh C <vignesh21@gmail.com> wrote:
> 
> Here is an updated version which includes registers to reset the memory
> context that is in-line with a recently committed patch at [1].

Thanks for updating patches ! They look good to me.

Just to confirm, would the other stuff (streamed_txns) that allocated under
CacheMemoryContext also leaks memory ? I think it's OK to change them
separately if it does but just would like to confirm if there is a risk.

> [1] -
> https://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=cfd6cbc
> f9be078fbdf9587014231a5772039b386

Best Regards,
Hou zj

pgsql-hackers by date:

Previous
From: Vladlen Popolitov
Date:
Subject: Re: Windows meson build
Next
From: "Andreas 'ads' Scherbaum"
Date:
Subject: Re: Crash: invalid DSA memory alloc request