Re: ldap: fix resource leak

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-hackers(at)postgresql(dot)org, Neil Conway <neilc(at)samurai(dot)com>
Cc: pgsql-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: ldap: fix resource leak
Date: 2006-11-05 04:34:52
Message-ID: 9974.1162701292@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Neil Conway <neilc(at)samurai(dot)com> writes:
> I fixed this by adding the appropriate ldap_unbind() calls in error
> control paths. An alternative would be to have a single place do the
> error handling, and jump to that via goto.

Perhaps use a PG_TRY construct?

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message mark 2006-11-05 06:15:51 PostgreSQL 8.2 (from CVS devel) first impressions
Previous Message Neil Conway 2006-11-05 00:43:37 ldap: fix resource leak

Browse pgsql-patches by date

  From Date Subject
Next Message Simon Riggs 2006-11-05 09:21:05 Re: Proposal: vacuum and autovacuum parameters to control freezing
Previous Message Neil Conway 2006-11-05 00:43:37 ldap: fix resource leak