[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>

2 files changed, 5 insertions, 18 deletions

diff --git a/include/net/fib_rules.h b/include/net/fib_rules.h
index e9a074cf59..4f47250712 100644
--- a/include/net/fib_rules.h
+++ b/include/net/fib_rules.h
@@ -102,7 +102,7 @@ static inline u32 frh_get_table(struct fib_rule_hdr *frh, struct nlattr **nla)
 }
 
 extern int fib_rules_register(struct net *, struct fib_rules_ops *);
-extern int fib_rules_unregister(struct net *, struct fib_rules_ops *);
+extern void fib_rules_unregister(struct net *, struct fib_rules_ops *);
 extern void                     fib_rules_cleanup_ops(struct fib_rules_ops *);
 
 extern int                      fib_rules_lookup(struct fib_rules_ops *,
diff --git a/net/core/fib_rules.c b/net/core/fib_rules.c
index c5f78fed68..541728aa2b 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)
 }
 EXPORT_SYMBOL_GPL(fib_rules_cleanup_ops);
 
-int fib_rules_unregister(struct net *net, struct fib_rules_ops *ops)
+void fib_rules_unregister(struct net *net, struct fib_rules_ops *ops)
 {
-        int err = 0;
-        struct fib_rules_ops *o;
 
         spin_lock(&net->rules_mod_lock);
-        list_for_each_entry(o, &net->rules_ops, list) {
-                if (o == ops) {
-                        list_del_rcu(&o->list);
-                        fib_rules_cleanup_ops(ops);
-                        goto out;
-                }
-        }
-
-        err = -ENOENT;
-out:
+        list_del_rcu(&ops->list);
+        fib_rules_cleanup_ops(ops);
         spin_unlock(&net->rules_mod_lock);
 
         synchronize_rcu();
-        if (!err)
-                release_net(net);
-
-        return err;
+        release_net(net);
 }
 
 EXPORT_SYMBOL_GPL(fib_rules_unregister);