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-23 21:26:26
Message-ID: 200206232126.g5NLQQB17524@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


Your patch has been added to the PostgreSQL unapplied patches list at:

http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

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

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-23 21:29:34 Re: Patch for memory leaks in index scan
Previous Message Bruce Momjian 2002-06-23 14:58:56 Re: Fix for fe-secure on FreeBSD/Alpha and FreeBSD/i386, v4.4