aboutsummaryrefslogtreecommitdiff
path: root/inet
diff options
context:
space:
mode:
authorCarlos O'Donell <carlos@redhat.com>2016-06-02 23:30:11 -0400
committerCarlos O'Donell <carlos@redhat.com>2016-06-02 23:52:06 -0400
commit1c1e7fb65828c99d6e0f0f3857089b559a0c8189 (patch)
treeec54f58a6ecbbc2380b9f1e7a8d6b1ab21a7c235 /inet
parentf06f3f05b48c72e2c9b0fa78671f94fd22d67da8 (diff)
downloadglibc-1c1e7fb65828c99d6e0f0f3857089b559a0c8189.zip
glibc-1c1e7fb65828c99d6e0f0f3857089b559a0c8189.tar.gz
glibc-1c1e7fb65828c99d6e0f0f3857089b559a0c8189.tar.bz2
Fix macro API for __USE_KERNEL_IPV6_DEFS.
The use of __USE_KERNEL_IPV6_DEFS with ifndef is bad practice per: https://sourceware.org/glibc/wiki/Wundef. This change moves it to use 'if' and always define the macro. Please note that this is not the only problem with this code. I have a series of fixes after this one to resolve breakage with this code and add regression tests for it via compile-only source testing (to be discussed in another thread). Unfortunately __USE_KERNEL_XATTR_DEFS is set by the kernel and not glibc, and uses 'define', so we can't fix that yet.
Diffstat (limited to 'inet')
-rw-r--r--inet/netinet/in.h14
1 files changed, 7 insertions, 7 deletions
diff --git a/inet/netinet/in.h b/inet/netinet/in.h
index 6122ab5..c801593 100644
--- a/inet/netinet/in.h
+++ b/inet/netinet/in.h
@@ -92,10 +92,10 @@ enum
IPPROTO_MAX
};
-/* If __USE_KERNEL_IPV6_DEFS is defined then the user has included the kernel
+/* If __USE_KERNEL_IPV6_DEFS is 1 then the user has included the kernel
network headers first and we should use those ABI-identical definitions
- instead of our own. */
-#ifndef __USE_KERNEL_IPV6_DEFS
+ instead of our own, otherwise 0. */
+#if !__USE_KERNEL_IPV6_DEFS
enum
{
IPPROTO_HOPOPTS = 0, /* IPv6 Hop-by-Hop options. */
@@ -206,7 +206,7 @@ enum
#define INADDR_ALLRTRS_GROUP ((in_addr_t) 0xe0000002) /* 224.0.0.2 */
#define INADDR_MAX_LOCAL_GROUP ((in_addr_t) 0xe00000ff) /* 224.0.0.255 */
-#ifndef __USE_KERNEL_IPV6_DEFS
+#if !__USE_KERNEL_IPV6_DEFS
/* IPv6 address */
struct in6_addr
{
@@ -249,7 +249,7 @@ struct sockaddr_in
sizeof (struct in_addr)];
};
-#ifndef __USE_KERNEL_IPV6_DEFS
+#if !__USE_KERNEL_IPV6_DEFS
/* Ditto, for IPv6. */
struct sockaddr_in6
{
@@ -285,7 +285,7 @@ struct ip_mreq_source
};
#endif
-#ifndef __USE_KERNEL_IPV6_DEFS
+#if !__USE_KERNEL_IPV6_DEFS
/* Likewise, for IPv6. */
struct ipv6_mreq
{
@@ -532,7 +532,7 @@ extern int bindresvport6 (int __sockfd, struct sockaddr_in6 *__sock_in)
#ifdef __USE_GNU
struct cmsghdr; /* Forward declaration. */
-#ifndef __USE_KERNEL_IPV6_DEFS
+#if !__USE_KERNEL_IPV6_DEFS
/* IPv6 packet information. */
struct in6_pktinfo
{