From: Bruce Momjian <pgman@candle.pha.pa.us>
To: Volkan YAZICI <yazicivo@ttnet.net.tr>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-interfaces@postgresql.org
Subject: Re: Receiving SIGPIPE for PQfinish(<broken-connection>)
Date: Mon, 24 Apr 2006 13:14:11 -0400 (EDT)
Message-ID: <200604241714.k3OHEBA10763@candle.pha.pa.us> (raw)
In-Reply-To: <20060424101705.GB259@alamut>
Volkan YAZICI wrote:
> On Apr 23 12:09, Bruce Momjian wrote:
> > Did you ever get this fixed?
>
> Actually, I couldn't reproduce same error on my current system -
> although, as I understand, none of the related source files were
> modified. (That failure was reported using Debian Sarge 3.1, so
> it could be caused by buggy debian libraries - like their buggy
> getpw..._r() family.)
>
> > Volkan YAZICI wrote:
> > > On Mar 12 11:18, Tom Lane wrote:
> > > > pqsecure_write should have disabled SIGPIPE already. You should look
> > > > into why that is seemingly not working.
> > >
> > > When I omit --enable-thread-safety, code doesn't handle in the case of
> > > a SIG_ERR return on the call to pqsignal() made from pqsecure_write().
> > >
> > > Furthermore, I still couldn't figure out why pqsignal() cannot handle
> > > SIGPIPE - despite pqsignal() doesn't return SIG_ERR. (A related small
> > > debug output is attached.)
>
> I don't have much experience with signal handling but if any hacker will
> ever get chance to take a look at SIG_ERR returning pqsignal() calls'
> handling, above problem (?) will be clarified.
>
> > > Everything works fine when thread safety is enabled.
>
> What are the reasons for thread-safety is not enabled by default? Is
> this because related system may not support thread locking to be used in
> libpq?
Good question. One issue is that not all operating systems will pass
the regession configure tests, so we could only support threading on
some platforms by default. If we get enough OS support, we can then
support it by default, and have a flag to disable it, like we do with
readline handling.
--
Bruce Momjian http://candle.pha.pa.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-interfaces@postgresql.org
Cc: pgman@candle.pha.pa.us, yazicivo@ttnet.net.tr, tgl@sss.pgh.pa.us
Subject: Re: Receiving SIGPIPE for PQfinish(<broken-connection>)
In-Reply-To: <200604241714.k3OHEBA10763@candle.pha.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox