net: hns: fix return value check in __lb_other_process()
authorYunjian Wang <wangyunjian@huawei.com>
Sat, 26 Dec 2020 08:10:05 +0000 (16:10 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 28 Dec 2020 22:57:35 +0000 (14:57 -0800)
The function skb_copy() could return NULL, the return value
need to be checked.

Fixes: b5996f11ea54 ("net: add Hisilicon Network Subsystem basic ethernet support")
Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns/hns_ethtool.c

index 7165da0..a6e3f07 100644 (file)
@@ -415,6 +415,10 @@ static void __lb_other_process(struct hns_nic_ring_data *ring_data,
        /* for mutl buffer*/
        new_skb = skb_copy(skb, GFP_ATOMIC);
        dev_kfree_skb_any(skb);
+       if (!new_skb) {
+               netdev_err(ndev, "skb alloc failed\n");
+               return;
+       }
        skb = new_skb;
 
        check_ok = 0;