Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 4a0a34b

Browse files
committed
In contrib/uuid-ossp, #include headers needed for ntohl() and ntohs().
Oversight in commit b8cc8f9. 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.
1 parent 7fa93ee commit 4a0a34b

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

contrib/uuid-ossp/uuid-ossp.c

+4
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@
1717
#include "utils/builtins.h"
1818
#include "utils/uuid.h"
1919

20+
/* for ntohl/htonl */
21+
#include <netinet/in.h>
22+
#include <arpa/inet.h>
23+
2024
/*
2125
* It's possible that there's more than one uuid.h header file present.
2226
* We expect configure to set the HAVE_ symbol for only the one we want.

0 commit comments

Comments
 (0)