diff options
author | Colin Ian King <colin.king@canonical.com> | 2016-10-18 21:39:28 +0300 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2016-12-14 19:18:17 +0300 |
commit | ea7ef2accdeaf825165cccd60b848765677bf1f2 (patch) | |
tree | 00e3f2b88b75a93163d337e002759dce98563bea /drivers/infiniband | |
parent | e37a79e5d4cac3831fac3d4afbf2461f56b4b7bd (diff) | |
download | linux-ea7ef2accdeaf825165cccd60b848765677bf1f2.tar.xz |
qedr: return -EINVAL if pd is null and avoid null ptr dereference
Currently, if pd is null then we hit a null pointer derference
on accessing pd->pd_id. Instead of just printing an error message
we should also return -EINVAL immediately.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r-- | drivers/infiniband/hw/qedr/verbs.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/qedr/verbs.c b/drivers/infiniband/hw/qedr/verbs.c index a61514296767..b2a0eb8f73d2 100644 --- a/drivers/infiniband/hw/qedr/verbs.c +++ b/drivers/infiniband/hw/qedr/verbs.c @@ -511,8 +511,10 @@ int qedr_dealloc_pd(struct ib_pd *ibpd) struct qedr_dev *dev = get_qedr_dev(ibpd->device); struct qedr_pd *pd = get_qedr_pd(ibpd); - if (!pd) + if (!pd) { pr_err("Invalid PD received in dealloc_pd\n"); + return -EINVAL; + } DP_DEBUG(dev, QEDR_MSG_INIT, "Deallocating PD %d\n", pd->pd_id); dev->ops->rdma_dealloc_pd(dev->rdma_ctx, pd->pd_id); |