On Fri, Jan 12, 2018 at 11:37:22AM +0900, Michael Paquier wrote: > I have redone my set of previous tests and can confirm that PQhost is > behaving as I would expect it should, and those results are the same as > yours.
if (conn->connhost != NULL && - conn->connhost[conn->whichhost].type != CHT_HOST_ADDRESS) + conn->connhost[conn->whichhost].host != NULL && + conn->connhost[conn->whichhost].host[0] != '\0') return conn->connhost[conn->whichhost].host; - else if (conn->pghost != NULL && conn->pghost[0] != '\0') - return conn->pghost; Upon further review, the second bit of the patch is making me itching. I think that you should not remove the second check which returns conn->pghost if a value is found in it. -- Michael
signature.asc
Description: PGP signature