Re: [HACKERS] Indicate disabled triggers in \d

From: Neil Conway <neilc(at)samurai(dot)com>
To: Brendan Jurd <direvus(at)gmail(dot)com>
Cc: David Fetter <david(at)fetter(dot)org>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] Indicate disabled triggers in \d
Date: 2006-11-11 00:16:53
Message-ID: 1163204213.5342.46.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Tue, 2006-11-07 at 16:21 +1100, Brendan Jurd wrote:
> Minor fix to the previous patch; result7 was not being cleared at the
> end of the block.

The patch still leaks result7 circa line 1400 (CVS HEAD). I didn't look
closely, but you probably also leak result7 circa line 1209, if result6
is NULL.

(Yeah, we definitely need to refactor describeOneTableDetails().)

-Neil

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-11-11 00:34:09 Re: PostgreSQL 8.2 (from CVS devel) first impressions
Previous Message Tom Lane 2006-11-10 22:32:52 Re: [HACKERS] Bug in WAL backup documentation

Browse pgsql-patches by date

  From Date Subject
Next Message Brendan Jurd 2006-11-11 02:38:13 Re: [HACKERS] Indicate disabled triggers in \d
Previous Message Tom Lane 2006-11-10 22:32:52 Re: [HACKERS] Bug in WAL backup documentation