diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-12-17 22:24:13 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-12-17 22:24:35 -0500 |
| commit | 53140bf22bc4b361836b68f08a2946a2fd2ab240 (patch) | |
| tree | 6c97ef2c1515ba395e7db21a17f0b2b65c7add5a | |
| parent | 595333ff493a3b17d82133a01cd64128bb6175b7 (diff) | |
| download | postgresql-53140bf22bc4b361836b68f08a2946a2fd2ab240.tar.gz postgresql-53140bf22bc4b361836b68f08a2946a2fd2ab240.zip | |
In contrib/uuid-ossp, #include headers needed for ntohl() and ntohs().
Oversight in commit b8cc8f947. I just noticed this causes compiler
warnings on FreeBSD, and it really ought to cause warnings elsewhere too:
all references I can find say that <arpa/inet.h> is required for these.
We have a lot of code elsewhere that thinks that both <netinet/in.h>
and <arpa/inet.h> should be included for these functions, so do it that
way here too, even though <arpa/inet.h> ought to be sufficient according
to the references I consulted.
Back-patch to 9.4 where the previous commit landed.
| -rw-r--r-- | contrib/uuid-ossp/uuid-ossp.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/contrib/uuid-ossp/uuid-ossp.c b/contrib/uuid-ossp/uuid-ossp.c index 06fd0c7daac..8437b7cf23e 100644 --- a/contrib/uuid-ossp/uuid-ossp.c +++ b/contrib/uuid-ossp/uuid-ossp.c @@ -17,6 +17,10 @@ #include "utils/builtins.h" #include "utils/uuid.h" +/* for ntohl/htonl */ +#include <netinet/in.h> +#include <arpa/inet.h> + /* * It's possible that there's more than one uuid.h header file present. * We expect configure to set the HAVE_ symbol for only the one we want. |