diff options
author | Leon Romanovsky <leonro@mellanox.com> | 2019-06-16 08:05:20 -0400 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2019-06-20 15:17:59 -0400 |
commit | 836a0fbb3e76f704ad65ddfb57f00725245e509b (patch) | |
tree | 30d961b48c7029a952e6eeb5077d76ce0e4a5221 /drivers/infiniband/sw/rdmavt/mr.c | |
parent | 89a6da3cb8f30ee0aeca924d84bef688f22f883e (diff) |
RDMA: Check umem pointer validity prior to release
Update ib_umem_release() to behave similarly to kfree() and allow
submitting NULL pointer as safe input to this function.
Fixes: a52c8e2469c3 ("RDMA: Clean destroy CQ in drivers do not return errors")
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/sw/rdmavt/mr.c')
-rw-r--r-- | drivers/infiniband/sw/rdmavt/mr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/infiniband/sw/rdmavt/mr.c b/drivers/infiniband/sw/rdmavt/mr.c index 54f3f9c27552..db800eb2b1f5 100644 --- a/drivers/infiniband/sw/rdmavt/mr.c +++ b/drivers/infiniband/sw/rdmavt/mr.c | |||
@@ -560,8 +560,7 @@ int rvt_dereg_mr(struct ib_mr *ibmr, struct ib_udata *udata) | |||
560 | if (ret) | 560 | if (ret) |
561 | goto out; | 561 | goto out; |
562 | rvt_deinit_mregion(&mr->mr); | 562 | rvt_deinit_mregion(&mr->mr); |
563 | if (mr->umem) | 563 | ib_umem_release(mr->umem); |
564 | ib_umem_release(mr->umem); | ||
565 | kfree(mr); | 564 | kfree(mr); |
566 | out: | 565 | out: |
567 | return ret; | 566 | return ret; |