Skip to contentSkip to conversation

: Don't export common symbols from ipfwadm.ko

PATCH In review Patrick McHardy 1 participant 1 message Oct 14, 2004

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

PATCH 2/5 : Don't export common symbols from ipfwadm.ko

ip_conntrack_tcp_update is exported from both ip_conntrack and ipfwadm, causing bogus module dependencies. This patch moves it to ip_conntrack_standalone. ip_conntrack_count doesn't need to be exported at all.

# This is a BitKeeper generated diff -Nru style patch. #
# ChangeSet
# 2004/10/14 00:57:06+02:00 [email protected]
# [NETFILTER]: Don't export common symbols from ipfwadm.ko
#

#   Exported kernel symbols ip_conntrack_count and ip_conntrack_tcp_update were
#   showing up both in ip_conntrack.ko and ipfwadm.ko, causing bogus dependencies
#   in modules.dep.

#
# Signed-off-by: Olaf Kirch <[email protected]>
# Signed-off-by: Patrick McHardy <[email protected]>
#

# net/ipv4/netfilter/ip_conntrack_standalone.c
#   2004/10/14 00:56:40+02:00 [email protected] +3 -0
#   [NETFILTER]: Don't export common symbols from ipfwadm.ko
#   
#   Exported kernel symbols ip_conntrack_count and ip_conntrack_tcp_update were
#   showing up both in ip_conntrack.ko and ipfwadm.ko, causing bogus dependencies
#   in modules.dep.
#   
#   Signed-off-by: Olaf Kirch <[email protected]>
#   Signed-off-by: Patrick McHardy <[email protected]>
# 
# net/ipv4/netfilter/ip_conntrack_proto_tcp.c
#   2004/10/14 00:56:39+02:00 [email protected] +0 -1
#   [NETFILTER]: Don't export common symbols from ipfwadm.ko
#   
#   Exported kernel symbols ip_conntrack_count and ip_conntrack_tcp_update were
#   showing up both in ip_conntrack.ko and ipfwadm.ko, causing bogus dependencies
#   in modules.dep.
#   
#   Signed-off-by: Olaf Kirch <[email protected]>
#   Signed-off-by: Patrick McHardy <[email protected]>
# 
# net/ipv4/netfilter/ip_conntrack_core.c
#   2004/10/14 00:56:39+02:00 [email protected] +0 -1
#   [NETFILTER]: Don't export common symbols from ipfwadm.ko
#   
#   Exported kernel symbols ip_conntrack_count and ip_conntrack_tcp_update were
#   showing up both in ip_conntrack.ko and ipfwadm.ko, causing bogus dependencies
#   in modules.dep.
#   
#   Signed-off-by: Olaf Kirch <[email protected]>
#   Signed-off-by: Patrick McHardy <[email protected]>
# 
diff -Nru a/net/ipv4/netfilter/ip_conntrack_core.c b/net/ipv4/netfilter/ip_conntrack_core.c
--- a/net/ipv4/netfilter/ip_conntrack_core.c	2004-10-14 01:36:39 +02:00
+++ b/net/ipv4/netfilter/ip_conntrack_core.c	2004-10-14 01:36:39 +02:00
@@ -62,7 +62,6 @@

/* ip_conntrack_standalone needs this */
atomic_t ip_conntrack_count = ATOMIC_INIT(0);
-EXPORT_SYMBOL(ip_conntrack_count);

void (*ip_conntrack_destroyed)(struct ip_conntrack *conntrack) = NULL; LIST_HEAD(ip_conntrack_expect_list);

net/ipv4/netfilter/ip_conntrack_proto_tcp.c +0 -1
diff -Nru a/net/ipv4/netfilter/ip_conntrack_proto_tcp.c b/net/ipv4/netfilter/ip_conntrack_proto_tcp.c
@@ -737,7 +737,6 @@
 	return 1;
 }
  
-EXPORT_SYMBOL(ip_conntrack_tcp_update);
 #endif
 
 #define	TH_FIN	0x01
net/ipv4/netfilter/ip_conntrack_standalone.c +3 -0
diff -Nru a/net/ipv4/netfilter/ip_conntrack_standalone.c b/net/ipv4/netfilter/ip_conntrack_standalone.c
@@ -903,3 +903,6 @@
 EXPORT_SYMBOL(ip_conntrack_untracked);
 EXPORT_SYMBOL_GPL(ip_conntrack_find_get);
 EXPORT_SYMBOL_GPL(ip_conntrack_put);
+#ifdef CONFIG_IP_NF_NAT_NEEDED
+EXPORT_SYMBOL(ip_conntrack_tcp_update);
+#endif