diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-11-27 00:10:04 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-11-27 00:10:04 +0000 |
commit | c2138f3caad74c904e86121b22ee63e8495618bf (patch) | |
tree | 1919e1f320e71d630cc955ab6033f3309f0f7387 /src/backend/executor/functions.c | |
parent | 9858a8c81cc1c433e273a55a278b757716201179 (diff) | |
download | postgresql-c2138f3caad74c904e86121b22ee63e8495618bf.tar.gz postgresql-c2138f3caad74c904e86121b22ee63e8495618bf.zip |
Fix minor memory leak introduced in recent SQL-functions hacking: the
DestReceiver created during postquel_start needs to be destroyed during
postquel_end. In a moment of brain fade I had assumed this would be taken
care of by FreeQueryDesc, but it's not (and shouldn't be).
Diffstat (limited to 'src/backend/executor/functions.c')
-rw-r--r-- | src/backend/executor/functions.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/backend/executor/functions.c b/src/backend/executor/functions.c index b34b90428ef..988f704ad72 100644 --- a/src/backend/executor/functions.c +++ b/src/backend/executor/functions.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/executor/functions.c,v 1.128 2008/10/31 21:07:55 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/executor/functions.c,v 1.129 2008/11/27 00:10:04 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -501,6 +501,8 @@ postquel_end(execution_state *es) PopActiveSnapshot(); } + (*es->qd->dest->rDestroy) (es->qd->dest); + FreeQueryDesc(es->qd); es->qd = NULL; } |