From: Leon Romanovsky <leo...@nvidia.com>

Add missed freeing previously allocated devlink object.

Fixes: a925b5e309c9 ("net/mlx5: Register mlx5 devices to auxiliary virtual bus")
Signed-off-by: Leon Romanovsky <leo...@nvidia.com>
Signed-off-by: Saeed Mahameed <sae...@nvidia.com>
---
 drivers/net/ethernet/mellanox/mlx5/core/main.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c 
b/drivers/net/ethernet/mellanox/mlx5/core/main.c
index c08315b51fd3..ca6f2fc39ea0 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/main.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c
@@ -1368,8 +1368,10 @@ static int init_one(struct pci_dev *pdev, const struct 
pci_device_id *id)
                         MLX5_COREDEV_VF : MLX5_COREDEV_PF;
 
        dev->priv.adev_idx = mlx5_adev_idx_alloc();
-       if (dev->priv.adev_idx < 0)
-               return dev->priv.adev_idx;
+       if (dev->priv.adev_idx < 0) {
+               err = dev->priv.adev_idx;
+               goto adev_init_err;
+       }
 
        err = mlx5_mdev_init(dev, prof_sel);
        if (err)
@@ -1403,6 +1405,7 @@ static int init_one(struct pci_dev *pdev, const struct 
pci_device_id *id)
        mlx5_mdev_uninit(dev);
 mdev_init_err:
        mlx5_adev_idx_free(dev->priv.adev_idx);
+adev_init_err:
        mlx5_devlink_free(devlink);
 
        return err;
-- 
2.26.2

Reply via email to