Re: rtree memory leak patch

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Kenneth Been <kennethb(at)telocity(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: rtree memory leak patch
Date: 2002-06-25 17:26:03
Message-ID: 200206251726.g5PHQ3P28284@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-patches


Patch applied. Thanks.

---------------------------------------------------------------------------

Kenneth Been wrote:
> Here is a patch for a memory leak in rtree.c, version 7.2.1 (in code
> that I submitted last year, alas).
>
> Ken
>
> *** postgresql-7.2.1/src/backend/access/rtree/rtree.c.orig Thu May
> 23 22:24:47 2002
> --- postgresql-7.2.1/src/backend/access/rtree/rtree.c Fri May 24
> 09:25:17 2002
> ***************
> *** 931,936 ****
> --- 931,938 ----
>
> PointerGetDatum(&size_alpha));
> FunctionCall2(&rtstate->sizeFn, union_dr,
>
> PointerGetDatum(&size_beta));
> + pfree(DatumGetPointer(union_dl));
> + pfree(DatumGetPointer(union_dr));
>
> diff = (size_alpha - size_l) - (size_beta -
> size_r);
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> message can get through to the mailing list cleanly
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2002-06-25 17:27:17 Re: several minor cleanups
Previous Message Rod Taylor 2002-06-25 16:18:14 Re: Dependency / Constraint patch