Remove unused helper_[find_get|put] functions
Subsystems (2)
Pablo Neira Ayuso
maintainerPATCH 5/10 Remove unused helper_[find_get|put] functions
Remove unused functions for conntrack application helpers
This patch depends on [PATCH 3/10]
Signed-off-by Pablo Neira Ayuso <[email protected]>
Signature · 1 line
The dawn of the fourth age of Linux firewalling is coming; a time of great struggle and heroic deeds -- J.Kadlecsik got inspired by J.Morris
[CONNTRACK] Remove unused helper_[find_get|put] functions
Remove unused functions for conntrack application helpers
Signed-off-by Pablo Neira Ayuso <[email protected]>
net/netfilter/nf_conntrack_core.c +3 -31
@@ -817,42 +817,14 @@ static inline int helper_cmp(const strucreturn nf_ct_tuple_mask_cmp (rtuple, &i->tuple, &i->mask); } -static struct nf_conntrack_helper * -__nf_ct_helper_find (const struct nf_conntrack_tuple *tuple) +struct nf_conntrack_helper * +nf_ct_helper_find (const struct nf_conntrack_tuple *tuple) {return LIST_FIND (&helpers, helper_cmp, struct nf_conntrack_helper *, tuple); } -struct nf_conntrack_helper * -nf_ct_helper_find_get (const struct nf_conntrack_tuple *tuple) -{ -struct nf_conntrack_helper *helper; - -/* need nf_conntrack_lock to assure that helper exists until -* try_module_get() is called */ -read_lock_bh (&nf_conntrack_lock); - - helper =__nf_ct_helper_find (tuple); -if (helper) { -/* need to increase module usage count to assure helper will -* not go away while the caller is e.g. busy putting a -* conntrack in the hash that uses the helper */ -if (!try_module_get (helper->me)) - helper =NULL ; - } - -read_unlock_bh (&nf_conntrack_lock); - -return helper; -} - -void nf_ct_helper_put (struct nf_conntrack_helper *helper) -{ -module_put (helper->me); -} -static struct nf_conn * __nf_conntrack_alloc(const struct nf_conntrack_tuple *orig, const struct nf_conntrack_tuple *repl,
@@ -885,7 +857,7 @@ __nf_conntrack_alloc(const struct nf_con/* FIXME: protect helper list per RCU */ read_lock_bh (&nf_conntrack_lock); - helper =__nf_ct_helper_find (repl); + helper =nf_ct_helper_find (repl);if (helper) features |=NF_CT_F_HELP ;read_unlock_bh (&nf_conntrack_lock);
include/net/netfilter/nf_conntrack.h +1 -2
@@ -221,8 +221,7 @@ extern void nf_ct_remove_expectations(stextern void nf_conntrack_flush (void );extern struct nf_conntrack_helper * -nf_ct_helper_find_get (const struct nf_conntrack_tuple *tuple); -extern void nf_ct_helper_put (struct nf_conntrack_helper *helper); +nf_ct_helper_find (const struct nf_conntrack_tuple *tuple); extern struct nf_conntrack_helper *
__nf_conntrack_helper_find_byname(const char *name);
net/netfilter/nf_conntrack_standalone.c +1 -2
@@ -889,8 +889,7 @@ EXPORT_SYMBOL(nf_conntrack_alloc);EXPORT_SYMBOL (nf_conntrack_free);EXPORT_SYMBOL (nf_conntrack_flush);EXPORT_SYMBOL (nf_ct_remove_expectations); -EXPORT_SYMBOL (nf_ct_helper_find_get); -EXPORT_SYMBOL (nf_ct_helper_put); +EXPORT_SYMBOL (nf_ct_helper_find);EXPORT_SYMBOL (__nf_conntrack_helper_find_byname);EXPORT_SYMBOL (__nf_conntrack_find);EXPORT_SYMBOL (nf_ct_unlink_expect);
net/ipv4/netfilter/ip_conntrack_core.c +4 -32
@@ -566,42 +566,14 @@ static inline int helper_cmp(const strucreturn ip_ct_tuple_mask_cmp (rtuple, &i->tuple, &i->mask); } -static struct ip_conntrack_helper * -__ip_conntrack_helper_find (const struct ip_conntrack_tuple *tuple) +struct ip_conntrack_helper * +ip_conntrack_helper_find (const struct ip_conntrack_tuple *tuple) {return LIST_FIND (&helpers, helper_cmp, struct ip_conntrack_helper *, tuple); } -struct ip_conntrack_helper * -ip_conntrack_helper_find_get (const struct ip_conntrack_tuple *tuple) -{ -struct ip_conntrack_helper *helper; - -/* need ip_conntrack_lock to assure that helper exists until -* try_module_get() is called */ -read_lock_bh (&ip_conntrack_lock); - - helper =__ip_conntrack_helper_find (tuple); -if (helper) { -/* need to increase module usage count to assure helper will -* not go away while the caller is e.g. busy putting a -* conntrack in the hash that uses the helper */ -if (!try_module_get (helper->me)) - helper =NULL ; - } - -read_unlock_bh (&ip_conntrack_lock); - -return helper; -} - -void ip_conntrack_helper_put (struct ip_conntrack_helper *helper) -{ -module_put (helper->me); -} -struct ip_conntrack_protocol *__ip_conntrack_proto_find (u_int8_t protocol) {
@@ -730,7 +702,7 @@ init_conntrack(struct ip_conntrack_tuplenf_conntrack_get (&conntrack->master->ct_general);CONNTRACK_STAT_INC (expect_new); }else { - conntrack->helper =__ip_conntrack_helper_find (&repl_tuple); + conntrack->helper =ip_conntrack_helper_find (&repl_tuple);CONNTRACK_STAT_INC (new); }
@@ -1055,7 +1027,7 @@ void ip_conntrack_alter_reply(struct ip_ conntrack->tuplehash[IP_CT_DIR_REPLY ].tuple = *newreply;if (!conntrack->master && conntrack->expecting ==0 ) - conntrack->helper =__ip_conntrack_helper_find (newreply); + conntrack->helper =ip_conntrack_helper_find (newreply);write_unlock_bh (&ip_conntrack_lock); }
net/ipv4/netfilter/ip_conntrack_standalone.c +1 -2
@@ -954,8 +954,7 @@ EXPORT_SYMBOL_GPL(ip_conntrack_hash_inseEXPORT_SYMBOL_GPL (ip_ct_remove_expectations); -EXPORT_SYMBOL_GPL (ip_conntrack_helper_find_get); -EXPORT_SYMBOL_GPL (ip_conntrack_helper_put); +EXPORT_SYMBOL_GPL (ip_conntrack_helper_find);EXPORT_SYMBOL_GPL (__ip_conntrack_helper_find_byname);EXPORT_SYMBOL_GPL (ip_conntrack_proto_find_get);
include/linux/netfilter_ipv4/ip_conntrack.h +1 -2
@@ -255,8 +255,7 @@ ip_ct_iterate_cleanup(int (*iter)(structextern struct ip_conntrack_helper *__ip_conntrack_helper_find_byname (const char *);extern struct ip_conntrack_helper * -ip_conntrack_helper_find_get (const struct ip_conntrack_tuple *tuple); -extern void ip_conntrack_helper_put (struct ip_conntrack_helper *helper); +ip_conntrack_helper_find (const struct ip_conntrack_tuple *tuple);extern struct ip_conntrack_protocol *__ip_conntrack_proto_find (u_int8_t protocol);