Skip to contentSkip to conversation

net/ipv4/netfilter/: fix SYSCTL=n compile

PATCH In review Adrian Bunk 2 participants 4 messages Jul 8, 2006 – Jul 10, 2006

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 net/ipv4/netfilter/: fix SYSCTL=n compile

This patch fixes the following compile error with CONFIG_SYSCTL=n introduced by commit 39a27a35c5c1b5be499a0576a35c45a011788bf8:

<-- snip -->

...
LD .tmp_vmlinux1
net/built-in.o: In function `tcp_error':
ip_conntrack_proto_tcp.c:(.text+0x77af6): undefined reference to ip_conntrack_checksum' net/built-in.o: In function udp_error':
ip_conntrack_proto_udp.c:(.text+0x78456): undefined reference to ip_conntrack_checksum' net/built-in.o: In function icmp_error':
ip_conntrack_proto_icmp.c:(.text+0x7868f): undefined reference to `ip_conntrack_checksum'
make: *** [.tmp_vmlinux1] Error 1

<-- snip -->

---

 net/ipv4/netfilter/ip_conntrack_standalone.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
net/ipv4/netfilter/ip_conntrack_standalone.c renamed from net/ipv4/netfilter/ip_conntrack_standalone.c.old +2 -2
@@ -534,6 +534,8 @@
 
 /* Sysctl support */
 
+int ip_conntrack_checksum = 1;
+
 #ifdef CONFIG_SYSCTL
 
 /* From ip_conntrack_core.c */
@@ -568,8 +570,6 @@
 static int log_invalid_proto_min = 0;
 static int log_invalid_proto_max = 255;
 
-int ip_conntrack_checksum = 1;
-
 static struct ctl_table_header *ip_ct_sysctl_header;
 
 static ctl_table ip_ct_sysctl_table[] = {
#1 lore raw
Quoted 2 lines · Adrian Bunk

This patch fixes the following compile error with CONFIG_SYSCTL=n introduced by commit 39a27a35c5c1b5be499a0576a35c45a011788bf8:

My fault I guess.

Quoted 11 lines

<-- snip -->

...
LD .tmp_vmlinux1
net/built-in.o: In function `tcp_error':
ip_conntrack_proto_tcp.c:(.text+0x77af6): undefined reference to ip_conntrack_checksum' net/built-in.o: In function udp_error':
ip_conntrack_proto_udp.c:(.text+0x78456): undefined reference to ip_conntrack_checksum' net/built-in.o: In function icmp_error':
ip_conntrack_proto_icmp.c:(.text+0x7868f): undefined reference to `ip_conntrack_checksum'
make: *** [.tmp_vmlinux1] Error 1

Thanks Adrian. Usually all bugs in ip_conntrack are duplicated in nf_conntrack, please update your patch to take care of that as well.

#2 lore raw

PATCH 1/1 netfilter: fix SYSCTL=n compile

Quoted 18 lines · Patrick McHardy
Quoted 2 lines · Adrian Bunk

This patch fixes the following compile error with CONFIG_SYSCTL=n introduced by commit 39a27a35c5c1b5be499a0576a35c45a011788bf8:

My fault I guess.

Quoted 11 lines

<-- snip -->

...
LD .tmp_vmlinux1
net/built-in.o: In function `tcp_error':
ip_conntrack_proto_tcp.c:(.text+0x77af6): undefined reference to ip_conntrack_checksum' net/built-in.o: In function udp_error':
ip_conntrack_proto_udp.c:(.text+0x78456): undefined reference to ip_conntrack_checksum' net/built-in.o: In function icmp_error':
ip_conntrack_proto_icmp.c:(.text+0x7868f): undefined reference to `ip_conntrack_checksum'
make: *** [.tmp_vmlinux1] Error 1

Quoted 2 lines

Thanks Adrian. Usually all bugs in ip_conntrack are duplicated in nf_conntrack, please update your patch to take care of that as well.

Correct, updated patch below.

cu
Adrian

<-- snip -->

This patch fixes the following compile errors with CONFIG_SYSCTL=n introduced by commit 39a27a35c5c1b5be499a0576a35c45a011788bf8:

<-- snip -->

...
LD .tmp_vmlinux1
net/built-in.o: In function `tcp_error':
ip_conntrack_proto_tcp.c:(.text+0x77af6): undefined reference to ip_conntrack_checksum' net/built-in.o: In function udp_error':
ip_conntrack_proto_udp.c:(.text+0x78456): undefined reference to ip_conntrack_checksum' net/built-in.o: In function icmp_error':
ip_conntrack_proto_icmp.c:(.text+0x7868f): undefined reference to `ip_conntrack_checksum'
make: *** [.tmp_vmlinux1] Error 1

<-- snip -->

...
LD .tmp_vmlinux1
net/built-in.o: In function `tcp_error':
nf_conntrack_proto_tcp.c:(.text+0x46b44): undefined reference to nf_conntrack_checksum' net/built-in.o: In function udp_error':
nf_conntrack_proto_udp.c:(.text+0x474f5): undefined reference to nf_conntrack_checksum' net/built-in.o: In function icmp_error':
nf_conntrack_proto_icmp.c:(.text+0x8e5f4): undefined reference to nf_conntrack_checksum' net/built-in.o: In function icmpv6_error':
nf_conntrack_proto_icmpv6.c:(.text+0xd3f54): undefined reference to `nf_conntrack_checksum'
make: *** [.tmp_vmlinux1] Error 1

<-- snip -->

---

 net/ipv4/netfilter/ip_conntrack_standalone.c |    4 ++--
 net/netfilter/nf_conntrack_standalone.c      |    4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
net/ipv4/netfilter/ip_conntrack_standalone.c renamed from net/ipv4/netfilter/ip_conntrack_standalone.c.old +2 -2
@@ -534,6 +534,8 @@
 
 /* Sysctl support */
 
+int ip_conntrack_checksum = 1;
+
 #ifdef CONFIG_SYSCTL
 
 /* From ip_conntrack_core.c */
@@ -568,8 +570,6 @@
 static int log_invalid_proto_min = 0;
 static int log_invalid_proto_max = 255;
 
-int ip_conntrack_checksum = 1;
-
 static struct ctl_table_header *ip_ct_sysctl_header;
 
 static ctl_table ip_ct_sysctl_table[] = {
net/netfilter/nf_conntrack_standalone.c renamed from net/netfilter/nf_conntrack_standalone.c.old +2 -2
@@ -428,6 +428,8 @@
 
 /* Sysctl support */
 
+int nf_conntrack_checksum = 1;
+
 #ifdef CONFIG_SYSCTL
 
 /* From nf_conntrack_core.c */
@@ -459,8 +461,6 @@
 static int log_invalid_proto_min = 0;
 static int log_invalid_proto_max = 255;
 
-int nf_conntrack_checksum = 1;
-
 static struct ctl_table_header *nf_ct_sysctl_header;
 
 static ctl_table nf_ct_sysctl_table[] = {
#3 lore raw

Re: [2.6 patch] netfilter: fix SYSCTL=n compile

Quoted 2 lines · Adrian Bunk

This patch fixes the following compile errors with CONFIG_SYSCTL=n introduced by commit 39a27a35c5c1b5be499a0576a35c45a011788bf8:

Thanks, applied.