bnxt_en: Call bnxt_ethtool_free() in bnxt_init_one() error path
[ Upstream commit 03400aaa69f916a376e11526cf591901a96a3a5c ]
bnxt_ethtool_init() may have allocated some memory and we need to
call bnxt_ethtool_free() to properly unwind if bnxt_init_one()
fails.
Fixes: 7c38091814
("bnxt_en: Refactor bnxt_init_one() and turn on TPA support on 57500 chips.")
Signed-off-by: Somnath Kotur <somnath.kotur@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
f8774be4dc
commit
c5d70dbc4d
@@ -12746,6 +12746,7 @@ init_err_pci_clean:
|
|||||||
bnxt_hwrm_func_drv_unrgtr(bp);
|
bnxt_hwrm_func_drv_unrgtr(bp);
|
||||||
bnxt_free_hwrm_short_cmd_req(bp);
|
bnxt_free_hwrm_short_cmd_req(bp);
|
||||||
bnxt_free_hwrm_resources(bp);
|
bnxt_free_hwrm_resources(bp);
|
||||||
|
bnxt_ethtool_free(bp);
|
||||||
kfree(bp->fw_health);
|
kfree(bp->fw_health);
|
||||||
bp->fw_health = NULL;
|
bp->fw_health = NULL;
|
||||||
bnxt_cleanup_pci(bp);
|
bnxt_cleanup_pci(bp);
|
||||||
|
Reference in New Issue
Block a user