diff options
author | Florian Westphal <fw@strlen.de> | 2017-03-07 12:45:04 +0100 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-03-08 19:01:23 +0100 |
commit | efc9b8e33b8b5ef890288758454ce62a1319c94a (patch) | |
tree | 9eee158b2ccf44e4e9c52f3166dd843d673048bc /net/bridge | |
parent | c7a72e3fdb5d77486ca3a0ac942c0a2e0d80d5bb (diff) | |
download | linux-stable-efc9b8e33b8b5ef890288758454ce62a1319c94a.tar.gz linux-stable-efc9b8e33b8b5ef890288758454ce62a1319c94a.tar.bz2 linux-stable-efc9b8e33b8b5ef890288758454ce62a1319c94a.zip |
netfilter: bridge: remove unneeded rcu_read_lock
as comment says, the function is always called with rcu read lock held.
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/bridge')
-rw-r--r-- | net/bridge/br_netfilter_hooks.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/bridge/br_netfilter_hooks.c b/net/bridge/br_netfilter_hooks.c index 95087e6e8258..52739e6c610e 100644 --- a/net/bridge/br_netfilter_hooks.c +++ b/net/bridge/br_netfilter_hooks.c @@ -1016,13 +1016,10 @@ int br_nf_hook_thresh(unsigned int hook, struct net *net, if (!elem) return okfn(net, sk, skb); - /* We may already have this, but read-locks nest anyway */ - rcu_read_lock(); nf_hook_state_init(&state, hook, NFPROTO_BRIDGE, indev, outdev, sk, net, okfn); ret = nf_hook_slow(skb, &state, elem); - rcu_read_unlock(); if (ret == 1) ret = okfn(net, sk, skb); |