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:13 -0500 |
| commit | 4a0a34b5b678f0292d3a85a85fb10c79c393be26 (patch) | |
| tree | 74f55eca3b3779e617977b7bb742e0ab7474ce93 | |
| parent | 7fa93eec4e0c9c3e801e3c51aa4bae3a38aaa218 (diff) | |
| download | postgresql-4a0a34b5b678f0292d3a85a85fb10c79c393be26.tar.gz postgresql-4a0a34b5b678f0292d3a85a85fb10c79c393be26.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 3e12bc4e96f..5d00f392ccc 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. |