Re: Plug minor memleak in pg_dump

Поиск
Список
Период
Сортировка
Искать
От
Daniel Gustafsson
Тема
Re: Plug minor memleak in pg_dump
Дата
Msg-id
F10CDF8A-CB7E-4D80-8FBB-1D57B443A03D@yesql.se
Ответ на
Список
Дерево обсуждения
Plug minor memleak in pg_dump gkokolatos@pm.me
Re: Plug minor memleak in pg_dump Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Re: Plug minor memleak in pg_dump Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Re: Plug minor memleak in pg_dump Daniel Gustafsson <daniel@yesql.se>
Re: Plug minor memleak in pg_dump Michael Paquier <michael@paquier.xyz>
Re: Plug minor memleak in pg_dump Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Re: Plug minor memleak in pg_dump Daniel Gustafsson <daniel@yesql.se>
> On 9 Feb 2022, at 03:56, Michael Paquier  wrote:
> 
> On Wed, Feb 02, 2022 at 10:06:13AM +0100, Daniel Gustafsson wrote:
>> The leak itself is clearly not something to worry about wrt memory pressure.
>> We do read into tmp and free it in other places in the same function though (as
>> you note above), so for code consistency alone this is worth doing IMO (and it
>> reduces the risk of static analyzers flagging this).
>> 
>> Unless objected to I will go ahead with getting this committed.
> 
> Looks like you forgot to apply that?

No, but I was distracted by other things leaving this on the TODO list.  It's
been pushed now.

--
Daniel Gustafsson		https://vmware.com/



В списке pgsql-hackers по дате отправления
От: Daniel Gustafsson
Дата:
Сообщение: Re: Refactoring SSL tests
От: Daniel Gustafsson
Дата:
FAQ