net: sched: fix warning in tcindex_alloc_perfect_hash
authorPavel Skripkin <paskripkin@gmail.com>
Fri, 25 Jun 2021 20:23:48 +0000 (23:23 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 28 Jun 2021 20:03:29 +0000 (13:03 -0700)
Syzbot reported warning in tcindex_alloc_perfect_hash. The problem
was in too big cp->hash, which triggers warning in kmalloc. Since
cp->hash comes from userspace, there is no need to warn if value
is not correct

Fixes: b9a24bb76bf6 ("net_sched: properly handle failure case of tcf_exts_init()")
Reported-and-tested-by: syzbot+1071ad60cd7df39fdadb@syzkaller.appspotmail.com
Signed-off-by: Pavel Skripkin <paskripkin@gmail.com>
Acked-by: Cong Wang <cong.wang@bytedance.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/cls_tcindex.c

index c4007b9..5b27453 100644 (file)
@@ -304,7 +304,7 @@ static int tcindex_alloc_perfect_hash(struct net *net, struct tcindex_data *cp)
        int i, err = 0;
 
        cp->perfect = kcalloc(cp->hash, sizeof(struct tcindex_filter_result),
-                             GFP_KERNEL);
+                             GFP_KERNEL | __GFP_NOWARN);
        if (!cp->perfect)
                return -ENOMEM;