Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 53f31209 authored by Yunjian Wang's avatar Yunjian Wang Committed by Greg Kroah-Hartman
Browse files

net: hns: fix return value check in __lb_other_process() 



[ Upstream commit 5ede3ada3da7f050519112b81badc058190b9f9f ]

The function skb_copy() could return NULL, the return value
need to be checked.

Fixes: b5996f11 ("net: add Hisilicon Network Subsystem basic ethernet support")
Signed-off-by: default avatarYunjian Wang <wangyunjian@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 85039b79

drivers/net/ethernet/hisilicon/hns/hns_ethtool.c

+4 −0
Original line number Diff line number Diff line
@@ -447,6 +447,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;