Sven Eckelmann
2013-Apr-18 09:17 UTC
[Bridge] [PATCH] bridge: Add missing include for in6_addr
ee07c6e7a6f8a25c18f0a6b18152fbd7499245f6 ("bridge: export multicast database via netlink") added "struct br_mdb_entry" which includes an element using the type "struct in6_addr". The file linux/if_bridge.h is part of the uapi and not defining this type in this file or its includes causes build failures in userspace applications. Signed-off-by: Sven Eckelmann <sven at open-mesh.com> --- include/uapi/linux/if_bridge.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/if_bridge.h b/include/uapi/linux/if_bridge.h index 2d70d79..8e3f287 100644 --- a/include/uapi/linux/if_bridge.h +++ b/include/uapi/linux/if_bridge.h @@ -14,6 +14,7 @@ #define _UAPI_LINUX_IF_BRIDGE_H #include <linux/types.h> +#include <linux/in6.h> #define SYSFS_BRIDGE_ATTR "bridge" #define SYSFS_BRIDGE_FDB "brforward" -- 1.8.2.1
Sven Eckelmann
2013-Apr-18 11:04 UTC
[Bridge] [PATCH] bridge: Add missing include for in6_addr
On Thursday 18 April 2013 02:17:38 Sven Eckelmann wrote:> ee07c6e7a6f8a25c18f0a6b18152fbd7499245f6 ("bridge: export multicast database > via netlink") added "struct br_mdb_entry" which includes an element using > the type "struct in6_addr". The file linux/if_bridge.h is part of the uapi > and not defining this type in this file or its includes causes build > failures in userspace applications.It seems this one was already dropped [1]. So, I think it will not be different this time. Just ignore it. Kind regards, Sven [1] http://lkml.indiana.edu/hypermail/linux/kernel/1301.1/03024.html -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. URL: <http://lists.linuxfoundation.org/pipermail/bridge/attachments/20130418/3a232358/attachment.sig>
David Miller
2013-Apr-18 17:51 UTC
[Bridge] [PATCH] bridge: Add missing include for in6_addr
From: Sven Eckelmann <sven at open-mesh.com> Date: Thu, 18 Apr 2013 11:17:38 +0200> ee07c6e7a6f8a25c18f0a6b18152fbd7499245f6 ("bridge: export multicast database > via netlink") added "struct br_mdb_entry" which includes an element using the > type "struct in6_addr". The file linux/if_bridge.h is part of the uapi and not > defining this type in this file or its includes causes build failures in > userspace applications. > > Signed-off-by: Sven Eckelmann <sven at open-mesh.com>This will conflict with the in6_addr defined by the libc headers, we're screwed either way here, so we might as well require userspace to include the libc header that defines this type.