Re: Fix tiny memory leaks
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: Masahiko Sawada <sawada.mshk@gmail.com>
Cc: Tofig Aliev <t.aliev@postgrespro.ru>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2024-12-09T20:04:16Z
Lists: pgsql-hackers
> On 9 Dec 2024, at 20:03, Daniel Gustafsson <daniel@yesql.se> wrote: > >> On 9 Dec 2024, at 19:52, Masahiko Sawada <sawada.mshk@gmail.com> wrote: >> >> On Fri, Dec 6, 2024 at 12:49 AM Daniel Gustafsson <daniel@yesql.se> wrote: >>> >>>> On 6 Dec 2024, at 09:38, Tofig Aliev <t.aliev@postgrespro.ru> wrote: >>> >>>> There is a memory leak in functions check_application_name() and check_cluster_name(). >>>> Functions are located in src/backend/commands/variable.c >>> >>> LGTM. >> >> LGTM. It seems commit a9d58bfe8a3a missed to fix this memory leak. > > Yeah. While fairly insignificant in nature, and there has been no complaints > until now, I'm preparing a backpatch down to REL_15_STABLE to keep the code in > sync for other backpatches to apply clean. Done, but to 16 and not 15 which was a typo in the above email. -- Daniel Gustafsson
Commits
-
Fix small memory leaks in GUC checks
- be9dac9afc25 16.7 landed
- 9add1bbfa665 17.3 landed
- 73a392d23696 18.0 landed
-
Fix tiny memory leaks
- a9d58bfe8a3a 16.0 cited