* [PATCH nf-next] netfilter: nf_dup{4,6}: fix build error when nf_conntrack disabled
@ 2015-09-02 18:54 Daniel Borkmann
2015-09-02 23:28 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Borkmann @ 2015-09-02 18:54 UTC (permalink / raw)
To: pablo; +Cc: fw, netfilter-devel, netdev, Daniel Borkmann
While testing various Kconfig options on another issue, I found that
the following one triggers as well on allmodconfig and nf_conntrack
disabled:
net/ipv4/netfilter/nf_dup_ipv4.c: In function ‘nf_dup_ipv4’:
net/ipv4/netfilter/nf_dup_ipv4.c:72:20: error: ‘nf_skb_duplicated’ undeclared (first use in this function)
if (this_cpu_read(nf_skb_duplicated))
[...]
net/ipv6/netfilter/nf_dup_ipv6.c: In function ‘nf_dup_ipv6’:
net/ipv6/netfilter/nf_dup_ipv6.c:66:20: error: ‘nf_skb_duplicated’ undeclared (first use in this function)
if (this_cpu_read(nf_skb_duplicated))
Fix it by including directly the header where it is defined.
Fixes: bbde9fc1824a ("netfilter: factor out packet duplication for IPv4/IPv6")
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
---
[ Don't know whether Dave wants to take it directly, or if it should
go via nf-next. I have one more build fix coming later tonight. Also
applies to net-next. ]
net/ipv4/netfilter/nf_dup_ipv4.c | 1 +
net/ipv6/netfilter/nf_dup_ipv6.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/net/ipv4/netfilter/nf_dup_ipv4.c b/net/ipv4/netfilter/nf_dup_ipv4.c
index b5bb375..2d79e6e 100644
--- a/net/ipv4/netfilter/nf_dup_ipv4.c
+++ b/net/ipv4/netfilter/nf_dup_ipv4.c
@@ -13,6 +13,7 @@
#include <linux/percpu.h>
#include <linux/route.h>
#include <linux/skbuff.h>
+#include <linux/netfilter.h>
#include <net/checksum.h>
#include <net/icmp.h>
#include <net/ip.h>
diff --git a/net/ipv6/netfilter/nf_dup_ipv6.c b/net/ipv6/netfilter/nf_dup_ipv6.c
index c5c87e9..c8ab626 100644
--- a/net/ipv6/netfilter/nf_dup_ipv6.c
+++ b/net/ipv6/netfilter/nf_dup_ipv6.c
@@ -11,6 +11,7 @@
#include <linux/module.h>
#include <linux/percpu.h>
#include <linux/skbuff.h>
+#include <linux/netfilter.h>
#include <net/ipv6.h>
#include <net/ip6_route.h>
#include <net/netfilter/ipv6/nf_dup_ipv6.h>
--
1.9.3
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH nf-next] netfilter: nf_dup{4,6}: fix build error when nf_conntrack disabled
2015-09-02 18:54 [PATCH nf-next] netfilter: nf_dup{4,6}: fix build error when nf_conntrack disabled Daniel Borkmann
@ 2015-09-02 23:28 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2015-09-02 23:28 UTC (permalink / raw)
To: daniel; +Cc: pablo, fw, netfilter-devel, netdev
From: Daniel Borkmann <daniel@iogearbox.net>
Date: Wed, 2 Sep 2015 20:54:02 +0200
> While testing various Kconfig options on another issue, I found that
> the following one triggers as well on allmodconfig and nf_conntrack
> disabled:
>
> net/ipv4/netfilter/nf_dup_ipv4.c: In function ‘nf_dup_ipv4’:
> net/ipv4/netfilter/nf_dup_ipv4.c:72:20: error: ‘nf_skb_duplicated’ undeclared (first use in this function)
> if (this_cpu_read(nf_skb_duplicated))
> [...]
> net/ipv6/netfilter/nf_dup_ipv6.c: In function ‘nf_dup_ipv6’:
> net/ipv6/netfilter/nf_dup_ipv6.c:66:20: error: ‘nf_skb_duplicated’ undeclared (first use in this function)
> if (this_cpu_read(nf_skb_duplicated))
>
> Fix it by including directly the header where it is defined.
>
> Fixes: bbde9fc1824a ("netfilter: factor out packet duplication for IPv4/IPv6")
> Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
I'll take this directly to simplify things.
Thanks Daniel.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-09-02 23:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-02 18:54 [PATCH nf-next] netfilter: nf_dup{4,6}: fix build error when nf_conntrack disabled Daniel Borkmann
2015-09-02 23:28 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).