Re: connect patch

2002-02-07 Thread Corinna Vinschen
On Thu, Feb 07, 2002 at 09:44:36AM -0500, Jason Tishler wrote: > Corinna, > > On Thu, Feb 07, 2002 at 02:56:25PM +0100, Corinna Vinschen wrote: > > On Wed, Feb 06, 2002 at 01:07:28PM -0500, Jason Tishler wrote: > > > Was this the right thing to do? > > > > The patch isn't correct since it now ca

Re: connect patch

2002-02-07 Thread Jason Tishler
Corinna, On Thu, Feb 07, 2002 at 02:56:25PM +0100, Corinna Vinschen wrote: > On Wed, Feb 06, 2002 at 01:07:28PM -0500, Jason Tishler wrote: > > Was this the right thing to do? > > The patch isn't correct since it now calls fdsock() twice which allocates > a new fhandler even if the line before a

Re: connect patch

2002-02-07 Thread Corinna Vinschen
On Wed, Feb 06, 2002 at 01:07:28PM -0500, Jason Tishler wrote: > The attached patch fixes a SEGV when getsockname () is called. This > problem can be tickled by the PostgreSQL 7.2 version of psql: > > http://archives.postgresql.org/pgsql-cygwin/2002-02/msg00012.php > > Note that I essential

Re: connect patch

2002-02-07 Thread Jason Tishler
On Wed, Feb 06, 2002 at 01:10:40PM -0500, Christopher Faylor wrote: > On Wed, Feb 06, 2002 at 01:07:28PM -0500, Jason Tishler wrote: > >Was this the right thing to do? > > I think so. It begs the question, though: do we need to apply this > patch elsewhere, too? I don't know. Maybe Corinna wou

Re: connect patch

2002-02-06 Thread Christopher Faylor
On Wed, Feb 06, 2002 at 01:07:28PM -0500, Jason Tishler wrote: >The attached patch fixes a SEGV when getsockname () is called. This >problem can be tickled by the PostgreSQL 7.2 version of psql: > >http://archives.postgresql.org/pgsql-cygwin/2002-02/msg00012.php > >Note that I essentially pla

connect patch

2002-02-06 Thread Jason Tishler
The attached patch fixes a SEGV when getsockname () is called. This problem can be tickled by the PostgreSQL 7.2 version of psql: http://archives.postgresql.org/pgsql-cygwin/2002-02/msg00012.php Note that I essentially plagiarized the following commit: http://cygwin.com/ml/cygwin-cvs/2