include/uapi/linux/netfilter_bridge.h: include if.h

Started by Mikko Rapeli on Feb 16, 2015 · 3 messages from 2 authors · Last reply Mar 11, 2015

Subsystems (2)

Maintainers (3)

STALE4261d

open the first message on its own page

Message details and related content
Fixes userspace compilation errors like:

error: field ‘in’ has incomplete type
struct in_addr in;

Signed-off-by: Mikko Rapeli <redacted>
---
 include/uapi/linux/netfilter_bridge.h | 1 +
 1 file changed, 1 insertion(+)
diff --git a/include/uapi/linux/netfilter_bridge.h b/include/uapi/linux/netfilter_bridge.h
index a5eda6d..5bb0528 100644
--- a/include/uapi/linux/netfilter_bridge.h
+++ b/include/uapi/linux/netfilter_bridge.h
@@ -4,6 +4,7 @@
 /* bridge-specific defines for netfilter. 
  */
 
+#include <linux/if.h>
 #include <linux/netfilter.h>
 #include <linux/if_ether.h>
 #include <linux/if_vlan.h>
-- 
2.1.4
Message details and related content
On Tuesday 2015-02-17 00:05, Mikko Rapeli wrote:
Fixes userspace compilation errors like:

error: field ‘in’ has incomplete type
struct in_addr in;

+#include <linux/if.h>
Patch 36/45 included linux/in.h instead of linux/if.h for addressing "in has incomplete
type". Should this be used here too?
Message details and related content
On Tue, Feb 17, 2015 at 01:02:50AM +0100, Jan Engelhardt wrote:
On Tuesday 2015-02-17 00:05, Mikko Rapeli wrote:
quoted
Fixes userspace compilation errors like:

error: field ‘in’ has incomplete type
struct in_addr in;

+#include <linux/if.h>
Patch 36/45 included linux/in.h instead of linux/if.h for addressing "in has incomplete
type". Should this be used here too?
It could, yes. It seems I cut some corners since this was hiding behind it:

In file included from ./linux/netfilter_bridge.h:11:0,
                 from ./linux/netfilter_bridge.c:1:
./linux/if_pppox.h:42:11: error: ‘IFNAMSIZ’ undeclared here (not in a function)
  char          dev[IFNAMSIZ];          /* Local device to use */

I'll fix these.

-Mikko
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help