bnxt_en: Call bnxt_ethtool_free() in bnxt_init_one() error path
[ Upstream commit03400aaa69] 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