diff options
| author |   Eran Ben Elisha <eranbe@mellanox.com> | 2017-11-20 09:58:01 +0200 |
|---|---|---|
| committer |   Saeed Mahameed <saeedm@mellanox.com> | 2018-01-12 02:01:47 +0200 |
| commit | 72f36be06138bdc11bdbe1f04e4a3e2637ea438d (patch) | |
| tree | ab975a43730639e212df98c5dccd97752c0de441 /drivers/infiniband/hw/mlx5 | |
| parent | net/mlx5: Fix memory leak in bad flow of mlx5_alloc_irq_vectors (diff) | |
| download | linux-dev-72f36be06138bdc11bdbe1f04e4a3e2637ea438d.tar.xz linux-dev-72f36be06138bdc11bdbe1f04e4a3e2637ea438d.zip | |
net/mlx5: Fix mlx5_get_uars_page to return error code
Change mlx5_get_uars_page to return ERR_PTR in case of
allocation failure. Change all callers accordingly to
check the IS_ERR(ptr) instead of NULL.
Fixes: 59211bd3b632 ("net/mlx5: Split the load/unload flow into hardware and software flows")
Signed-off-by: Eran Ben Elisha <eranbe@mellanox.com>
Signed-off-by: Eugenia Emantayev <eugenia@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
Diffstat (limited to 'drivers/infiniband/hw/mlx5')
| -rw-r--r-- | drivers/infiniband/hw/mlx5/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c index 00cb184fa027..262c1aa2e028 100644 --- a/drivers/infiniband/hw/mlx5/main.c +++ b/drivers/infiniband/hw/mlx5/main.c @@ -4160,7 +4160,7 @@ static void *mlx5_ib_add(struct mlx5_core_dev *mdev) goto err_cnt; dev->mdev->priv.uar = mlx5_get_uars_page(dev->mdev); - if (!dev->mdev->priv.uar) + if (IS_ERR(dev->mdev->priv.uar)) goto err_cong; err = mlx5_alloc_bfreg(dev->mdev, &dev->bfreg, false, false); |