Re: Large Objects buffer leak patch(es) - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: Large Objects buffer leak patch(es)
Date
Msg-id 199807210417.AAA21493@candle.pha.pa.us
Whole thread Raw
In response to Large Objects buffer leak patch(es)  (<andre@via.ecp.fr>)
List pgsql-hackers
Patch applied.

>
>     Hi all.
>
> As nobody answered my previous post, I decided to correct PostgreSQL
> buffer leaks caused by large objects methods.
>
> Theses buffer leaks are caused by indexes that are kept open between
> calls. Outside a transaction, the backend detects them as buffer leaks; it
> sends a NOTICE, and frees them. This sometimes cause a segmentation fault
> (at least on Linux). These indexes are initialized on the first
> lo_read/lo_write/lo_tell call, and (normally) closed on a lo_close call.
> Thus the buffer leaks appear when lo direct access functions are used, and
> not with lo_import/lo_export functions (libpq version calls lo_close
> before ending the command, and the backend version uses another path).
>
> The included patches (against recent snapshot, and against 6.3.2) cause
> indexes to be closed on transaction end (that is on explicit 'END'
> statment, or on command termination outside trasaction blocks), thus
> preventing the buffer leaks while increasing performance inside
> transactions. Some (all?) 'classic' memory leaks are also removed.
>
> I hope it will be ok.


--
Bruce Momjian                          |  830 Blythe Avenue
maillist@candle.pha.pa.us              |  Drexel Hill, Pennsylvania 19026
  +  If your life is a hard drive,     |  (610) 353-9879(w)
  +  Christ can be your backup.        |  (610) 853-3000(h)

pgsql-hackers by date:

Previous
From: The Hermit Hacker
Date:
Subject: Re: [HACKERS] cidr
Next
From: darcy@druid.net (D'Arcy J.M. Cain)
Date:
Subject: Re: [HACKERS] cidr