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:44 -0500 |
| commit | 20c27fdfd3af8da68837ab1cbea08100619f91b6 (patch) | |
| tree | 86d31a7470e4b703d1d8c8e954d5ad3f00b23209 | |
| parent | 779325478e7e867c636c3e1ad1e5df734e7549e5 (diff) | |
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 9e9905bfdef..9bf30259114 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. |