summaryrefslogtreecommitdiffstats
path: root/net/bridge
diff options
context:
space:
mode:
authorGao feng <gaofeng@cn.fujitsu.com>2013-02-18 16:59:10 +0000
committerPablo Neira Ayuso <pablo@netfilter.org>2013-03-15 11:56:09 +0100
commitfa900b9cf5a574cc66cc9b50749999d8b6de6ed8 (patch)
tree702cb2d8fbff1fd0fc2d2913e2c9cd624f5586b7 /net/bridge
parentd00bd3d4fba89e6f7ffb94a5f9274cce49dc84a7 (diff)
downloadop-kernel-dev-fa900b9cf5a574cc66cc9b50749999d8b6de6ed8.zip
op-kernel-dev-fa900b9cf5a574cc66cc9b50749999d8b6de6ed8.tar.gz
netfilter: ebt_ulog: remove unnecessary spin lock protection
No need for spinlock to protect the netlink skb in the ebt_ulog_fini path. We are sure there is noone using it at that stage. Signed-off-by: Gao feng <gaofeng@cn.fujitsu.com> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/bridge')
-rw-r--r--net/bridge/netfilter/ebt_ulog.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/net/bridge/netfilter/ebt_ulog.c b/net/bridge/netfilter/ebt_ulog.c
index 3bf43f7..442b032 100644
--- a/net/bridge/netfilter/ebt_ulog.c
+++ b/net/bridge/netfilter/ebt_ulog.c
@@ -319,12 +319,11 @@ static void __exit ebt_ulog_fini(void)
for (i = 0; i < EBT_ULOG_MAXNLGROUPS; i++) {
ub = &ulog_buffers[i];
del_timer(&ub->timer);
- spin_lock_bh(&ub->lock);
+
if (ub->skb) {
kfree_skb(ub->skb);
ub->skb = NULL;
}
- spin_unlock_bh(&ub->lock);
}
netlink_kernel_release(ebtulognl);
}
OpenPOWER on IntegriCloud