diff options
author | Shiraz Saleem <shiraz.saleem@intel.com> | 2019-03-28 11:49:45 -0500 |
---|---|---|
committer | Jason Gunthorpe <jgg@mellanox.com> | 2019-03-28 14:13:27 -0300 |
commit | 41d34865b24c6a0b594b0a69bfe9ea56dff5abcd (patch) | |
tree | f8c21b06f980dd2585c4c13d6cb3bb668a3a0fea /drivers | |
parent | 5f818d676ac455bbc812ffaaf5bf780be5465114 (diff) | |
download | linux-stable-41d34865b24c6a0b594b0a69bfe9ea56dff5abcd.tar.gz linux-stable-41d34865b24c6a0b594b0a69bfe9ea56dff5abcd.tar.bz2 linux-stable-41d34865b24c6a0b594b0a69bfe9ea56dff5abcd.zip |
RDMA/mthca: Use correct sizing on buffers holding page DMA addresses
The buffer that holds the page DMA addresses is sized off umem->nmap.
This can potentially cause out of bound accesses on the PBL array when
iterating the umem DMA-mapped SGL. This is because if umem pages are
combined, umem->nmap can be much lower than the number of system pages
in umem.
Use ib_umem_num_pages() to size this buffer.
Signed-off-by: Shiraz Saleem <shiraz.saleem@intel.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/hw/mthca/mthca_provider.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mthca/mthca_provider.c b/drivers/infiniband/hw/mthca/mthca_provider.c index d063d7a37762..35c3119726bb 100644 --- a/drivers/infiniband/hw/mthca/mthca_provider.c +++ b/drivers/infiniband/hw/mthca/mthca_provider.c @@ -914,7 +914,7 @@ static struct ib_mr *mthca_reg_user_mr(struct ib_pd *pd, u64 start, u64 length, goto err; } - n = mr->umem->nmap; + n = ib_umem_num_pages(mr->umem); mr->mtt = mthca_alloc_mtt(dev, n); if (IS_ERR(mr->mtt)) { |