Check that filter is not NULL before passing it to tcf_walker->fn()
callback. This can happen when mall_change() failed to offload filter to
hardware.

Fixes: ed76f5edccc9 ("net: sched: protect filter_chain list with 
filter_chain_lock mutex")
Reported-by: Ido Schimmel <ido...@mellanox.com>
Tested-by: Ido Schimmel <ido...@mellanox.com>
Signed-off-by: Vlad Buslov <vla...@mellanox.com>
---
 net/sched/cls_matchall.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/net/sched/cls_matchall.c b/net/sched/cls_matchall.c
index a37137430e61..1f9d481b0fbb 100644
--- a/net/sched/cls_matchall.c
+++ b/net/sched/cls_matchall.c
@@ -247,6 +247,9 @@ static void mall_walk(struct tcf_proto *tp, struct 
tcf_walker *arg,
 
        if (arg->count < arg->skip)
                goto skip;
+
+       if (!head)
+               return;
        if (arg->fn(tp, head, arg) < 0)
                arg->stop = 1;
 skip:
-- 
2.13.6

Reply via email to