Re: Fix resource leak (src/backend/libpq/be-secure-common.c)

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Fix resource leak (src/backend/libpq/be-secure-common.c)
Дата
Msg-id F8483DF1-268F-44D3-8D3A-CCB333E2D81F@yesql.se
обсуждение исходный текст
Ответ на Re: Fix resource leak (src/backend/libpq/be-secure-common.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Ответы Re: Fix resource leak (src/backend/libpq/be-secure-common.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Список pgsql-hackers
On 10 Apr 2024, at 20:31, Ranier Vilela <ranier.vf@gmail.com> wrote:

Em ter., 2 de abr. de 2024 às 15:31, Daniel Gustafsson <daniel@yesql.se> escreveu:
> On 2 Apr 2024, at 20:13, Ranier Vilela <ranier.vf@gmail.com> wrote:

> Fix by freeing the pointer, like pclose_check (src/common/exec.c) similar case.

Off the cuff, seems reasonable when loglevel is LOG.

Per Coverity.

Another case of resource leak, when loglevel is LOG.
In the function shell_archive_file (src/backend/archive/shell_archive.c)
The pointer *xlogarchcmd*  is not freed.

Thanks, I'll have a look.  I've left this for post-freeze on purpose to not
cause unnecessary rebasing.  Will take a look over the next few days unless
beaten to it.

--
Daniel Gustafsson

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Jelte Fennema-Nio
Дата:
Сообщение: Re: psql: Greatly speed up "\d tablename" when not using regexes
Следующее
От: Jelte Fennema-Nio
Дата:
Сообщение: Re: psql: Greatly speed up "\d tablename" when not using regexes