[PATCH] [net-next] net: ip, diag: include net/inet_sock.h

Subsystems: networking [general], the rest

STALE3621d

3 messages, 3 authors, 2016-10-29 · open the first message on its own page

[PATCH] [net-next] net: ip, diag: include net/inet_sock.h

From: Arnd Bergmann <arnd@arndb.de>
Date: 2016-10-25 15:53:59

The newly added raw_diag.c fails to build in some configurations
unless we include this header:

In file included from net/ipv4/raw_diag.c:6:0:
include/net/raw.h:71:21: error: field 'inet' has incomplete type
net/ipv4/raw_diag.c: In function 'raw_diag_dump':
net/ipv4/raw_diag.c:166:29: error: implicit declaration of function 'inet_sk'

Fixes: 432490f9d455 ("net: ip, diag -- Add diag interface for raw sockets")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 net/ipv4/raw_diag.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/net/ipv4/raw_diag.c b/net/ipv4/raw_diag.c
index ef3bea061b75..be930908bcf9 100644
--- a/net/ipv4/raw_diag.c
+++ b/net/ipv4/raw_diag.c
@@ -3,6 +3,7 @@
 #include <linux/inet_diag.h>
 #include <linux/sock_diag.h>
 
+#include <net/inet_sock.h>
 #include <net/raw.h>
 #include <net/rawv6.h>
 
-- 
2.9.0

Re: [PATCH] [net-next] net: ip, diag: include net/inet_sock.h

From: Cyrill Gorcunov <hidden>
Date: 2016-10-25 16:06:08

On Tue, Oct 25, 2016 at 05:53:22PM +0200, Arnd Bergmann wrote:
The newly added raw_diag.c fails to build in some configurations
unless we include this header:

In file included from net/ipv4/raw_diag.c:6:0:
include/net/raw.h:71:21: error: field 'inet' has incomplete type
net/ipv4/raw_diag.c: In function 'raw_diag_dump':
net/ipv4/raw_diag.c:166:29: error: implicit declaration of function 'inet_sk'

Fixes: 432490f9d455 ("net: ip, diag -- Add diag interface for raw sockets")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Thank you! And sorry for this, it didn't trigger on my testing.

Re: [PATCH] [net-next] net: ip, diag: include net/inet_sock.h

From: David Miller <davem@davemloft.net>
Date: 2016-10-29 18:53:13

From: Arnd Bergmann <arnd@arndb.de>
Date: Tue, 25 Oct 2016 17:53:22 +0200
The newly added raw_diag.c fails to build in some configurations
unless we include this header:

In file included from net/ipv4/raw_diag.c:6:0:
include/net/raw.h:71:21: error: field 'inet' has incomplete type
net/ipv4/raw_diag.c: In function 'raw_diag_dump':
net/ipv4/raw_diag.c:166:29: error: implicit declaration of function 'inet_sk'

Fixes: 432490f9d455 ("net: ip, diag -- Add diag interface for raw sockets")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Applied, thanks Arnd.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help