diff options
| author | Jason Gunthorpe <[email protected]> | 2020-11-24 23:34:33 +0000 |
|---|---|---|
| committer | Jason Gunthorpe <[email protected]> | 2020-11-26 19:49:04 +0000 |
| commit | d0b7721c5e0d940a514fbff40b9d7bae92735f7a (patch) | |
| tree | e98eebcea0b6826e52980eda9c6c67caa8c299e1 /drivers/infiniband/hw/mlx5/devx.c | |
| parent | RDMA/hns: Add UD support for HIP09 (diff) | |
| download | kernel-d0b7721c5e0d940a514fbff40b9d7bae92735f7a.tar.gz kernel-d0b7721c5e0d940a514fbff40b9d7bae92735f7a.zip | |
RDMA/mlx5: Check for ERR_PTR from uverbs_zalloc()
The return code from uverbs_zalloc() was wrongly checked, it is ERR_PTR
not NULL like other allocators:
drivers/infiniband/hw/mlx5/devx.c:2110 devx_umem_reg_cmd_alloc() warn: passing zero to 'PTR_ERR'
Fixes: 878f7b31c3a7 ("RDMA/mlx5: Use ib_umem_find_best_pgsz() for devx")
Link: https://lore.kernel.org/r/[email protected]
Reported-by: kernel test robot <[email protected]>
Acked-by: Leon Romanovsky <[email protected]>
Signed-off-by: Jason Gunthorpe <[email protected]>
Diffstat (limited to 'drivers/infiniband/hw/mlx5/devx.c')
| -rw-r--r-- | drivers/infiniband/hw/mlx5/devx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mlx5/devx.c b/drivers/infiniband/hw/mlx5/devx.c index 7c3eefba6197..ad0173f62c0e 100644 --- a/drivers/infiniband/hw/mlx5/devx.c +++ b/drivers/infiniband/hw/mlx5/devx.c @@ -2106,7 +2106,7 @@ static int devx_umem_reg_cmd_alloc(struct mlx5_ib_dev *dev, (MLX5_ST_SZ_BYTES(mtt) * ib_umem_num_dma_blocks(obj->umem, page_size)); cmd->in = uverbs_zalloc(attrs, cmd->inlen); - if (!cmd->in) + if (IS_ERR(cmd->in)) return PTR_ERR(cmd->in); umem = MLX5_ADDR_OF(create_umem_in, cmd->in, umem); |