diff options
author | Liping Zhang <zlpnobody@gmail.com> | 2017-02-18 10:35:47 +0800 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-02-21 13:45:47 +0100 |
commit | 4eba8b78e1390f46828e00d6fa4d651c3d219894 (patch) | |
tree | 0ef5f78b44a71b8461608b35d3b9afc01d676e5d /net/netfilter/nfnetlink.c | |
parent | ba896a05ad9375912ccebdac9623aab97845600f (diff) | |
download | op-kernel-dev-4eba8b78e1390f46828e00d6fa4d651c3d219894.zip op-kernel-dev-4eba8b78e1390f46828e00d6fa4d651c3d219894.tar.gz |
netfilter: nfnetlink: remove static declaration from err_list
Otherwise, different subsys will race to access the err_list, with holding
the different nfnl_lock(subsys_id).
But this will not happen now, since ->call_batch is only implemented by
nftables, so the err_list is protected by nfnl_lock(NFNL_SUBSYS_NFTABLES).
Signed-off-by: Liping Zhang <zlpnobody@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net/netfilter/nfnetlink.c')
-rw-r--r-- | net/netfilter/nfnetlink.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/nfnetlink.c b/net/netfilter/nfnetlink.c index a09fa9f..6fa4484 100644 --- a/net/netfilter/nfnetlink.c +++ b/net/netfilter/nfnetlink.c @@ -279,7 +279,7 @@ static void nfnetlink_rcv_batch(struct sk_buff *skb, struct nlmsghdr *nlh, struct net *net = sock_net(skb->sk); const struct nfnetlink_subsystem *ss; const struct nfnl_callback *nc; - static LIST_HEAD(err_list); + LIST_HEAD(err_list); u32 status; int err; |