aboutsummaryrefslogtreecommitdiffstats
path: root/net/core
diff options
context:
space:
mode:
authorDenis V. Lunev <den@openvz.org>2008-01-15 01:59:30 -0500
committerDavid S. Miller <davem@davemloft.net>2008-01-28 18:02:09 -0500
commit72132c1b6c7a7605cdfde86a9f58b6ca2b2195a6 (patch)
tree35b13af8135ad136800f39b9a7aff40082123878 /net/core
parent2db82b534bd52b349f1b2ab3e63aa40ca0e466ab (diff)
[IPV4]: fib_rules_unregister is essentially void.
fib_rules_unregister is called only after successful register and the return code is never checked. Signed-off-by: Denis V. Lunev <den@openvz.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r--net/core/fib_rules.c21
1 files changed, 4 insertions, 17 deletions
diff --git a/net/core/fib_rules.c b/net/core/fib_rules.c
index c5f78fed6885..541728aa2bae 100644
--- a/net/core/fib_rules.c
+++ b/net/core/fib_rules.c
@@ -112,29 +112,16 @@ void fib_rules_cleanup_ops(struct fib_rules_ops *ops)
112} 112}
113EXPORT_SYMBOL_GPL(fib_rules_cleanup_ops); 113EXPORT_SYMBOL_GPL(fib_rules_cleanup_ops);
114 114
115int fib_rules_unregister(struct net *net, struct fib_rules_ops *ops) 115void fib_rules_unregister(struct net *net, struct fib_rules_ops *ops)
116{ 116{
117 int err = 0;
118 struct fib_rules_ops *o;
119 117
120 spin_lock(&net->rules_mod_lock); 118 spin_lock(&net->rules_mod_lock);
121 list_for_each_entry(o, &net->rules_ops, list) { 119 list_del_rcu(&ops->list);
122 if (o == ops) { 120 fib_rules_cleanup_ops(ops);
123 list_del_rcu(&o->list);
124 fib_rules_cleanup_ops(ops);
125 goto out;
126 }
127 }
128
129 err = -ENOENT;
130out:
131 spin_unlock(&net->rules_mod_lock); 121 spin_unlock(&net->rules_mod_lock);
132 122
133 synchronize_rcu(); 123 synchronize_rcu();
134 if (!err) 124 release_net(net);
135 release_net(net);
136
137 return err;
138} 125}
139 126
140EXPORT_SYMBOL_GPL(fib_rules_unregister); 127EXPORT_SYMBOL_GPL(fib_rules_unregister);