Re: Large Objects buffer leak patch(es)

From: Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us>
To: andre(at)via(dot)ecp(dot)fr
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Large Objects buffer leak patch(es)
Date: 1998-07-21 04:17:49
Message-ID: 199807210417.AAA21493@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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(at)candle(dot)pha(dot)pa(dot)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)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message D'Arcy J.M. Cain 1998-07-21 04:20:36 Re: [HACKERS] cidr
Previous Message The Hermit Hacker 1998-07-21 04:06:47 Re: [HACKERS] cidr