summaryrefslogtreecommitdiffstats
path: root/net/sched
diff options
context:
space:
mode:
authorJiri Pirko <jiri@mellanox.com>2017-05-22 16:46:13 +0200
committerDavid S. Miller <davem@davemloft.net>2017-05-22 20:54:16 +0200
commit2d76b2f8b54abd16225cd80afca36ed43f113c41 (patch)
tree4bdb6eb967b730de091e58a5fd8635272216c146 /net/sched
parentvsock: use new wait API for vsock_stream_sendmsg() (diff)
downloadlinux-2d76b2f8b54abd16225cd80afca36ed43f113c41.tar.xz
linux-2d76b2f8b54abd16225cd80afca36ed43f113c41.zip
net: sched: cls_matchall: fix null pointer dereference
Since the head is guaranteed by the check above to be null, the call_rcu would explode. Remove the previously logically dead code that was made logically very much alive and kicking. Fixes: 985538eee06f ("net/sched: remove redundant null check on head") Signed-off-by: Jiri Pirko <jiri@mellanox.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
-rw-r--r--net/sched/cls_matchall.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/net/sched/cls_matchall.c b/net/sched/cls_matchall.c
index dee469fed967..51859b8edd7e 100644
--- a/net/sched/cls_matchall.c
+++ b/net/sched/cls_matchall.c
@@ -203,7 +203,6 @@ static int mall_change(struct net *net, struct sk_buff *in_skb,
*arg = (unsigned long) head;
rcu_assign_pointer(tp->root, new);
- call_rcu(&head->rcu, mall_destroy_rcu);
return 0;
err_replace_hw_filter: