Re: [PATCHES] ldap: fix resource leak

Поиск
Список
Период
Сортировка
От Neil Conway
Тема Re: [PATCHES] ldap: fix resource leak
Дата
Msg-id 1162770185.5692.341.camel@localhost.localdomain
обсуждение исходный текст
Ответ на Re: [PATCHES] ldap: fix resource leak  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCHES] ldap: fix resource leak  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Sat, 2006-11-04 at 23:34 -0500, Tom Lane wrote:
> Perhaps use a PG_TRY construct?

At least for the existing code, this doesn't work well: the function
exits early via ereport(LOG) and then "return STATUS_ERROR;", so AFAICS
there isn't an easy way to simplify the existing error handling logic
via PG_TRY.

Note that this is related to a more general problem: if *any* backend
function allocates a resource that needs to be manually cleaned up, it
probably ought to be using a PG_TRY block. Otherwise, the resource will
be leaked on elog(ERROR). I wouldn't be surprised if various parts of
the backend neglected to get this right. However, in this particular
case, I didn't bother doing this, since it didn't seem likely that
anything we're going to invoke will report errors via elog. One could
make an argument for doing, for the sake of correctness/paranoia...

-Neil



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

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: [PATCHES] Writing WAL for relcache invalidation:pg_internal.init
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [PATCHES] ldap: fix resource leak