cls_matchall: fix tcf_unbind_filter missing
authorHangbin Liu <liuhangbin@gmail.com>
Tue, 14 Aug 2018 09:28:26 +0000 (17:28 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Aug 2018 05:47:15 +0000 (07:47 +0200)
[ Upstream commit a51c76b4dfb30496dc65396a957ef0f06af7fb22 ]

Fix tcf_unbind_filter missing in cls_matchall as this will trigger
WARN_ON() in cbq_destroy_class().

Fixes: fd62d9f5c575f ("net/sched: matchall: Fix configuration race")
Reported-by: Li Shuang <shuali@redhat.com>
Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
Acked-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
net/sched/cls_matchall.c

index e75fb65..61ddfba 100644 (file)
@@ -94,6 +94,8 @@ static bool mall_destroy(struct tcf_proto *tp, bool force)
        if (!head)
                return true;
 
+       tcf_unbind_filter(tp, &head->res);
+
        if (tc_should_offload(dev, tp, head->flags))
                mall_destroy_hw_filter(tp, head, (unsigned long) head);