Thread: Receiving SIGPIPE for PQfinish()
Hi,
I've a problem when calling PQfinish() on a broken connection.
Breakpoint 1, main () at finish.c:9 9 PGconn *conn = PQconnectdb("dbname=test"); (gdb) n 11
if (PQstatus(conn) != CONNECTION_OK) (gdb) n 18 PQfinish(conn);
Shutting down postmaster in here (or restarting on another port).
(gdb) n Program received signal SIGPIPE, Broken pipe. 0xffffe410 in ?? () (gdb) bt #0 0xffffe410 in ?? () #1
0xbfafd368in ?? () #2 0xb7fe8308 in ?? () from ../usr/lib/libpq.so.4 #3 0xbfafd340 in ?? () #4 0xb7f68321 in send ()
from/lib/libc.so.6 #5 0xb7fe24a3 in pqsecure_write (conn=0xffffffe0, ptr=0xffffffe0, len=4294967264) at
fe-secure.c:480#6 0xb7fdbc9a in pqSendSome (conn=0x804a008, len=5) at fe-misc.c:757 #7 0xb7fd5448 in closePGconn
(conn=0x804a008)at fe-connect.c:1973 #8 0xb7fd5475 in PQfinish (conn=0x804a008) at fe-connect.c:2033 #9 0x080485f1 in
main() at finish.c:18
Should I handle SIGPIPE each time I call PQfinish() manually or should
PQfinish() be responsible for this task?
Regards.
Volkan YAZICI <yazicivo@ttnet.net.tr> writes:
> I've a problem when calling PQfinish() on a broken connection.
pqsecure_write should have disabled SIGPIPE already. You should look
into why that is seemingly not working.
regards, tom lane
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.) Everything works fine when thread safety is enabled. Regards.
Attachment
Did you ever get this fixed? --------------------------------------------------------------------------- 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.) > > Everything works fine when thread safety is enabled. > > > Regards. [ Attachment, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 5: don't forget to increase your free space map settings -- Bruce Momjian http://candle.pha.pa.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. +
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? Regards.
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. +