Re: Memory leak in CachememoryContext - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Memory leak in CachememoryContext
Date
Msg-id 20230424135541.phtdpfftovykfsti@alvherre.pgsql
Whole thread Raw
In response to Re: Memory leak in CachememoryContext  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Memory leak in CachememoryContext
List pgsql-hackers
On 2023-Apr-21, Tom Lane wrote:

> (Note to self: we can't remove the cast_hash_context field in
> back branches for fear of causing an ABI break for pldebugger.
> But we can leave it unused, I think.)

Hmm, we can leave it unused in our code, but it still needs to be
initialized to some valid memory context anyway; otherwise hypothetical
code that uses it would still crash.  This seems halfway obvious, but
since the submitted patch doesn't have this part, I thought better to
point it out.

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/



pgsql-hackers by date:

Previous
From: Dave Cramer
Date:
Subject: Re: Request for comment on setting binary format output per session
Next
From: Peter Eisentraut
Date:
Subject: Re: run pgindent on a regular basis / scripted manner