Commit a44490ab authored Dec 07, 2022 by Yuan Can Committed by Greg Kroah-Hartman Jan 18, 2023

drivers: net: qlcnic: Fix potential memory leak in qlcnic_sriov_init()



[ Upstream commit 01de1123322e4fe1bbd0fcdf0982511b55519c03 ]

If vp alloc failed in qlcnic_sriov_init(), all previously allocated vp
needs to be freed.

Fixes: f197a7aa ("qlcnic: VF-PF communication channel implementation")
Signed-off-by: Yuan Can <yuancan@huawei.com>
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 429370c4

drivers/net/ethernet/qlogic/qlcnic/qlcnic_sriov_common.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -222,6 +222,8 @@ int qlcnic_sriov_init(struct qlcnic_adapter *adapter, int num_vfs)
		return 0;

		qlcnic_destroy_async_wq:
		while (i--)
		kfree(sriov->vf_info[i].vp);
		destroy_workqueue(bc->bc_async_wq);

		qlcnic_destroy_trans_wq: