From 7a9bf93e1ca5ffd207da22bce03dfbe1024cd668 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 16 Apr 2014 13:21:09 -0400 Subject: [PATCH] Use AF_UNSPEC not PF_UNSPEC in getaddrinfo calls. According to the Single Unix Spec and assorted man pages, you're supposed to use the constants named AF_xxx when setting ai_family for a getaddrinfo call. In a few places we were using PF_xxx instead. Use of PF_xxx appears to be an ancient BSD convention that was not adopted by later standardization. On BSD and most later Unixen, it doesn't matter much because those constants have equivalent values anyway; but nonetheless this code is not per spec. In the same vein, replace PF_INET by AF_INET in one socket() call, which wasn't even consistent with the other socket() call in the same function let alone the remainder of our code. Per investigation of a Cygwin trouble report from Marco Atzeri. It's probably a long shot that this will fix his issue, but it's wrong in any case. --- src/backend/libpq/hba.c | 2 +- src/backend/postmaster/pgstat.c | 2 +- src/bin/initdb/initdb.c | 2 +- src/port/pipe.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/backend/libpq/hba.c b/src/backend/libpq/hba.c index b679e946538..5762fad3efb 100644 --- a/src/backend/libpq/hba.c +++ b/src/backend/libpq/hba.c @@ -744,7 +744,7 @@ parse_hba_line(List *line, int line_num, HbaLine *parsedline) /* Get the IP address either way */ hints.ai_flags = AI_NUMERICHOST; - hints.ai_family = PF_UNSPEC; + hints.ai_family = AF_UNSPEC; hints.ai_socktype = 0; hints.ai_protocol = 0; hints.ai_addrlen = 0; diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c index cf9492ed3fa..b12a0d86ec9 100644 --- a/src/backend/postmaster/pgstat.c +++ b/src/backend/postmaster/pgstat.c @@ -309,7 +309,7 @@ pgstat_init(void) * Create the UDP socket for sending and receiving statistic messages */ hints.ai_flags = AI_PASSIVE; - hints.ai_family = PF_UNSPEC; + hints.ai_family = AF_UNSPEC; hints.ai_socktype = SOCK_DGRAM; hints.ai_protocol = 0; hints.ai_addrlen = 0; diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.c index 3b34baf083d..4ddd94e3bba 100644 --- a/src/bin/initdb/initdb.c +++ b/src/bin/initdb/initdb.c @@ -1238,7 +1238,7 @@ setup_config(void) /* for best results, this code should match parse_hba() */ hints.ai_flags = AI_NUMERICHOST; - hints.ai_family = PF_UNSPEC; + hints.ai_family = AF_UNSPEC; hints.ai_socktype = 0; hints.ai_protocol = 0; hints.ai_addrlen = 0; diff --git a/src/port/pipe.c b/src/port/pipe.c index a6df1a4ee1e..631af393483 100644 --- a/src/port/pipe.c +++ b/src/port/pipe.c @@ -55,7 +55,7 @@ pgpipe(int handles[2]) closesocket(s); return -1; } - if ((handles[1] = socket(PF_INET, SOCK_STREAM, 0)) == INVALID_SOCKET) + if ((handles[1] = socket(AF_INET, SOCK_STREAM, 0)) == INVALID_SOCKET) { ereport(LOG, (errmsg_internal("pgpipe failed to create socket 2: %ui", WSAGetLastError()))); closesocket(s); -- 2.39.5