Content-Length: 261714 | pFad | http://github.com/postgrespro/postgres_cluster/commit/4a0a34b5b678f0292d3a85a85fb10c79c393be26

A0 In contrib/uuid-ossp, #include headers needed for ntohl() and ntohs(). · postgrespro/postgres_cluster@4a0a34b · GitHub
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

Lines changed: 4 additions & 0 deletions
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)








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgrespro/postgres_cluster/commit/4a0a34b5b678f0292d3a85a85fb10c79c393be26

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy