netfilter: nf_ct_ecache: refactor notifier registration
authorTony Zelenoff <antonz@parallels.com>
Thu, 8 Mar 2012 23:35:39 +0000 (23:35 +0000)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 8 May 2012 17:17:23 +0000 (19:17 +0200)
* ret variable initialization removed as useless
* similar code strings concatenated and functions code
  flow became more plain

Signed-off-by: Tony Zelenoff <antonz@parallels.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_conntrack_ecache.c

index b924f3a49a8e5c40cd4b71e38e5f70cbe2c6c2c3..e7be79e640de0397ac9f5e5aaab00faf1f2c8730 100644 (file)
@@ -84,7 +84,7 @@ EXPORT_SYMBOL_GPL(nf_ct_deliver_cached_events);
 int nf_conntrack_register_notifier(struct net *net,
                                   struct nf_ct_event_notifier *new)
 {
-       int ret = 0;
+       int ret;
        struct nf_ct_event_notifier *notify;
 
        mutex_lock(&nf_ct_ecache_mutex);
@@ -95,8 +95,7 @@ int nf_conntrack_register_notifier(struct net *net,
                goto out_unlock;
        }
        rcu_assign_pointer(net->ct.nf_conntrack_event_cb, new);
-       mutex_unlock(&nf_ct_ecache_mutex);
-       return ret;
+       ret = 0;
 
 out_unlock:
        mutex_unlock(&nf_ct_ecache_mutex);
@@ -121,7 +120,7 @@ EXPORT_SYMBOL_GPL(nf_conntrack_unregister_notifier);
 int nf_ct_expect_register_notifier(struct net *net,
                                   struct nf_exp_event_notifier *new)
 {
-       int ret = 0;
+       int ret;
        struct nf_exp_event_notifier *notify;
 
        mutex_lock(&nf_ct_ecache_mutex);
@@ -132,8 +131,7 @@ int nf_ct_expect_register_notifier(struct net *net,
                goto out_unlock;
        }
        rcu_assign_pointer(net->ct.nf_expect_event_cb, new);
-       mutex_unlock(&nf_ct_ecache_mutex);
-       return ret;
+       ret = 0;
 
 out_unlock:
        mutex_unlock(&nf_ct_ecache_mutex);