Re: [HACKERS] 6.4.1 release - Mailing list pgsql-hackers

From Tatsuo Ishii
Subject Re: [HACKERS] 6.4.1 release
Date
Msg-id 199812131114.UAA00951@ext16.sra.co.jp
Whole thread Raw
In response to Re: [HACKERS] 6.4.1 release  (Bruce Momjian <maillist@candle.pha.pa.us>)
List pgsql-hackers
> > > > I think at least large object stuffs should be fixed(just a "select
> > > > lo_import('/foo/bar')" easily kills backend) before 6.4.1. I've been
> > > > looking into codes for sometime but have not found complete fixes yet.
> > > 
> > > I thought we already had a large object fix in the two trees already?
> > 
> > So you fixed inv_api.c?  I got cvs header in REL6_4 tree (FreeBSD
> > 2.2.6-RELEASE). Is this the latest one?
> > 
> > *        $Header: /usr/local/cvsroot/pgsql/src/backend/storage/large_object/in\
> > v_api.c,v 1.41 1998/10/06 03:55:43 momjian Exp $
> > 
> > Following is a backend-crashing example. Any idea?
> > 
> > (/tmp/html.tar.gz is a 102458 bytes long file)
> > 
> > > select lo_import('/tmp/html.tar.gz');
> > blank
> >          1: lo_import   (typeid = 26, len = 4, typmod = -1, byval = t)
> >         ----
> 
> Fixed.  Since I re-designed the heap access API, the bug was crystal
> clear as soon as I looked at the code.  Scarry when I can figure out the
> backend code so quickly.
> 
> Patch applied to both trees.

Thanks. Your patches definitely fix the problem on
FreeBSD. Unfotunately it does not help Solaris/Sparc. I'll look into
more on that platform.
---
Tatsuo Ishii


pgsql-hackers by date:

Previous
From: "Oliver Elphick"
Date:
Subject: Re: [HACKERS] postgresql/c.h typedefs Size as 'unsigned int' (fwd)
Next
From: Thomas Good
Date:
Subject: Re: [HACKERS] ecpg man page