Re: Fix handling of invalid sockets returned by PQsocket() - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Fix handling of invalid sockets returned by PQsocket()
Date
Msg-id 20160217165804.GA873990@alvherre.pgsql
Whole thread Raw
In response to Fix handling of invalid sockets returned by PQsocket()  (Michael Paquier <michael.paquier@gmail.com>)
Responses Re: Fix handling of invalid sockets returned by PQsocket()
List pgsql-hackers
Michael Paquier wrote:
> Hi all,
> 
> After looking at Alvaro's message mentioning the handling of
> PQsocket() for invalid sockets, I just had a look by curiosity at
> other calls of this routine, and found a couple of issues:
> 1) In vacuumdb.c, init_slot() does not check for the return value of PQsocket():
> slot->sock = PQsocket(conn);
> 2) In isolationtester.c, try_complete_step() should do the same.
> 3) In pg_recvlogical.c for StreamLogicalLog() I am spotting the same problem.
> I guess those ones should be fixed as well, no?

I patched pgbench to use PQerrorMessage rather than strerror(errno).  I
think your patch should do the same.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Martin Liška
Date:
Subject: Re: [PATCH] Code refactoring related to -fsanitize=use-after-scope
Next
From: Joe Conway
Date:
Subject: Re: exposing pg_controldata and pg_config as functions