From: | Japin Li <japinli(at)hotmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Mark Dilger <mark(dot)dilger(at)enterprisedb(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Memory leak fix in psql |
Date: | 2022-07-21 06:43:03 |
Message-ID: | MEYP282MB16699E39A1FE9145A25EEBADB6919@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, 21 Jul 2022 at 14:23, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> On 2022-Jul-21, Japin Li wrote:
>
>> On Thu, 21 Jul 2022 at 09:48, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
>> > We are talking about 256 bytes being leaked in each loop when a
>> > validation pattern or when a query fails, so I don't see a strong
>> > argument in manipulating 10~14 more than necessary for this amount of
>> > memory. The contents of describe.c are the same for v15 though, and
>> > we are still in beta on REL_15_STABLE, so I have applied the patch
>> > down to v15, adding what Alvaro has sent on top of the rest.
>>
>> Thanks for the explanation! IMO, we could ignore v10-13 branches, however,
>> we should backpatch to v14 which also uses the validateSQLNamePattern()
>> function leading to a memory leak.
>
> I'd agree in principle, but in practice the commit from 15 does not apply
> cleanly on 14 -- there's a ton of conflicts.
I attached a patch for v14 [1] based on master, if you want to apply it,
please consider reviewing it.
[1] https://www.postgresql.org/message-id/MEYP282MB166974D3883A1A6E25B5FDAEB68E9%40MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM
--
Regrads,
Japin Li.
ChengDu WenWu Information Technology Co.,Ltd.
From | Date | Subject | |
---|---|---|---|
Next Message | Erik Rijkers | 2022-07-21 07:09:47 | Re: Schema variables - new implementation for Postgres 15 |
Previous Message | Alvaro Herrera | 2022-07-21 06:23:45 | Re: Memory leak fix in psql |