database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-11-03 09:13:20 +03:00
Code Activity

Fix portability issues in new src/port/inet_net_ntop.c file.

Browse Source 
1. Don't #include postgres.h in a frontend build.

2. Don't assume that the backend's symbol PGSQL_AF_INET6 has anything to do
with the constant that will be used by system library functions (because,
in point of fact, it usually doesn't).  Fortunately, PGSQL_AF_INET is equal
to AF_INET, so we can just cater for both sets of values in one case
construct without fear of conflict.
This commit is contained in:
Tom Lane
2010-11-26 17:59:10 -05:00
parent 55109313f9
commit 3840bc0847
1 changed files with 24 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
src/port/inet_net_ntop.c
View File

@@ -21,14 +21,28 @@
static const char rcsid[] = "Id: inet_net_ntop.c,v 1.1.2.2 2004/03/09 09:17:27 marka Exp $"; static const char rcsid[] = "Id: inet_net_ntop.c,v 1.1.2.2 2004/03/09 09:17:27 marka Exp $";
#endif #endif
#ifndef FRONTEND
#include "postgres.h" #include "postgres.h"
#else
#include "postgres_fe.h"
#endif
#include <sys/types.h> #include <sys/types.h>
#include <sys/socket.h> #include <sys/socket.h>
#include <netinet/in.h> #include <netinet/in.h>
#include <arpa/inet.h> #include <arpa/inet.h>
#ifndef FRONTEND
#include "utils/inet.h" #include "utils/inet.h"
#else
/*
* In a frontend build, we can't include inet.h, but we still need to have
* sensible definitions of these two constants. Note that inet_net_ntop()
* assumes that PGSQL_AF_INET is equal to AF_INET.
*/
#define PGSQL_AF_INET (AF_INET + 0)
#define PGSQL_AF_INET6 (AF_INET + 1)
#endif
#define NS_IN6ADDRSZ 16 #define NS_IN6ADDRSZ 16
@@ -63,11 +77,21 @@ static char *inet_net_ntop_ipv6(const u_char *src, int bits,
char * char *
inet_net_ntop(int af, const void *src, int bits, char *dst, size_t size) inet_net_ntop(int af, const void *src, int bits, char *dst, size_t size)
{ {
/*
* We need to cover both the address family constants used by the PG
* inet type (PGSQL_AF_INET and PGSQL_AF_INET6) and those used by the
* system libraries (AF_INET and AF_INET6). We can safely assume
* PGSQL_AF_INET == AF_INET, but the INET6 constants are very likely
* to be different. If AF_INET6 isn't defined, silently ignore it.
*/
switch (af) switch (af)
{ {
case PGSQL_AF_INET: case PGSQL_AF_INET:
return (inet_net_ntop_ipv4(src, bits, dst, size)); return (inet_net_ntop_ipv4(src, bits, dst, size));
case PGSQL_AF_INET6: case PGSQL_AF_INET6:
#if defined(AF_INET6) && AF_INET6 != PGSQL_AF_INET6
case AF_INET6:
#endif
return (inet_net_ntop_ipv6(src, bits, dst, size)); return (inet_net_ntop_ipv6(src, bits, dst, size));
default: default:
errno = EAFNOSUPPORT; errno = EAFNOSUPPORT;
@@ -272,4 +296,3 @@ inet_net_ntop_ipv6(const u_char *src, int bits, char *dst, size_t size)
strcpy(dst, tmp); strcpy(dst, tmp);
return (dst); return (dst);
} }