* [PATCH] Fix ipchains/ipfw modules
@ 2004-09-24 14:44 Andi Kleen
2004-09-24 15:15 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 3+ messages in thread
From: Andi Kleen @ 2004-09-24 14:44 UTC (permalink / raw)
To: netdev, coreteam
ipchains didn't load anymore because it had a undefined symbol,
which was satisfied by ipconntrack. This caused ipconntrack to be
loaded first by modprobe, but the second ipconntrack init in ipchains
would fail, causing the ipchains load to fail.
ipfw had the same problem.
Declare the missing variable.
Signed-off-by: Andi Kleen <ak@muc.de>
diff -u linux/net/ipv4/netfilter/ip_fw_compat.c-IPC linux/net/ipv4/netfilter/ip_fw_compat.c
--- linux/net/ipv4/netfilter/ip_fw_compat.c-IPC 2004-06-16 14:07:34.000000000 +0200
+++ linux/net/ipv4/netfilter/ip_fw_compat.c 2004-09-24 15:56:55.000000000 +0200
@@ -28,6 +28,8 @@
static struct firewall_ops *fwops;
+unsigned int ip_ct_log_invalid;
+
#ifdef CONFIG_IP_VS
/* From ip_vs_core.c */
extern unsigned int
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix ipchains/ipfw modules
2004-09-24 14:44 [PATCH] Fix ipchains/ipfw modules Andi Kleen
@ 2004-09-24 15:15 ` Arnaldo Carvalho de Melo
2004-09-24 18:07 ` David S. Miller
0 siblings, 1 reply; 3+ messages in thread
From: Arnaldo Carvalho de Melo @ 2004-09-24 15:15 UTC (permalink / raw)
To: Andi Kleen; +Cc: netdev, coreteam
Patrick McHardy fixed this in a different way in the "Re: [PATCH] Warn
people that ipchains and ipfwadm are going away." lkml thread :-)
His explanation:
"Fixed by this patch. The conntrack protocols need ip_ct_log_invalid
which is defined in ip_conntrack_standalone, so ip_conntrack is
loaded automatically before ipchains. This patch moves it over to
ip_conntrack_core."
- Arnaldo
Andi Kleen wrote:
> ipchains didn't load anymore because it had a undefined symbol,
> which was satisfied by ipconntrack. This caused ipconntrack to be
> loaded first by modprobe, but the second ipconntrack init in ipchains
> would fail, causing the ipchains load to fail.
>
> ipfw had the same problem.
>
> Declare the missing variable.
>
> Signed-off-by: Andi Kleen <ak@muc.de>
>
> diff -u linux/net/ipv4/netfilter/ip_fw_compat.c-IPC linux/net/ipv4/netfilter/ip_fw_compat.c
> --- linux/net/ipv4/netfilter/ip_fw_compat.c-IPC 2004-06-16 14:07:34.000000000 +0200
> +++ linux/net/ipv4/netfilter/ip_fw_compat.c 2004-09-24 15:56:55.000000000 +0200
> @@ -28,6 +28,8 @@
>
> static struct firewall_ops *fwops;
>
> +unsigned int ip_ct_log_invalid;
> +
> #ifdef CONFIG_IP_VS
> /* From ip_vs_core.c */
> extern unsigned int
>
>
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix ipchains/ipfw modules
2004-09-24 15:15 ` Arnaldo Carvalho de Melo
@ 2004-09-24 18:07 ` David S. Miller
0 siblings, 0 replies; 3+ messages in thread
From: David S. Miller @ 2004-09-24 18:07 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo; +Cc: ak, netdev, coreteam
On Fri, 24 Sep 2004 12:15:08 -0300
Arnaldo Carvalho de Melo <acme@conectiva.com.br> wrote:
> Patrick McHardy fixed this in a different way in the "Re: [PATCH] Warn
> people that ipchains and ipfwadm are going away." lkml thread :-)
>
> His explanation:
>
> "Fixed by this patch. The conntrack protocols need ip_ct_log_invalid
> which is defined in ip_conntrack_standalone, so ip_conntrack is
> loaded automatically before ipchains. This patch moves it over to
> ip_conntrack_core."
Yes, and that fix is on it's way to Linus today some time.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2004-09-24 18:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-09-24 14:44 [PATCH] Fix ipchains/ipfw modules Andi Kleen
2004-09-24 15:15 ` Arnaldo Carvalho de Melo
2004-09-24 18:07 ` David S. 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).