summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorFlorian Westphal <fw@strlen.de>2016-01-28 13:16:59 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2016-02-01 00:15:28 +0100
commit53c520c2ab79e9f3765d24116ab54f6d5b3cd563 (patch)
tree3c42bbbfa5e67ed24001dfd68f242b4c135c5ce5 /net
parent7c7bdf35991bb8f7cfaeaf22ea3a2f2d1967c166 (diff)
downloadop-kernel-dev-53c520c2ab79e9f3765d24116ab54f6d5b3cd563.zip
op-kernel-dev-53c520c2ab79e9f3765d24116ab54f6d5b3cd563.tar.gz
netfilter: cttimeout: fix deadlock due to erroneous unlock/lock conversion
The spin_unlock call should have been left as-is, revert. Fixes: b16c29191dc89bd ("netfilter: nf_conntrack: use safer way to lock all buckets") Reported-by: kernel test robot <fengguang.wu@intel.com> Signed-off-by: Florian Westphal <fw@strlen.de> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net')
-rw-r--r--net/netfilter/nfnetlink_cttimeout.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/nfnetlink_cttimeout.c b/net/netfilter/nfnetlink_cttimeout.c
index 94837d2..2671b9d 100644
--- a/net/netfilter/nfnetlink_cttimeout.c
+++ b/net/netfilter/nfnetlink_cttimeout.c
@@ -312,7 +312,7 @@ static void ctnl_untimeout(struct net *net, struct ctnl_timeout *timeout)
hlist_nulls_for_each_entry(h, nn, &net->ct.hash[i], hnnode)
untimeout(h, timeout);
}
- nf_conntrack_lock(&nf_conntrack_locks[i % CONNTRACK_LOCKS]);
+ spin_unlock(&nf_conntrack_locks[i % CONNTRACK_LOCKS]);
}
local_bh_enable();
}
OpenPOWER on IntegriCloud