summaryrefslogtreecommitdiffstats
path: root/drivers/infiniband/hw/hns/hns_roce_cq.c
diff options
context:
space:
mode:
authorLijun Ou2016-09-20 18:07:08 +0200
committerDoug Ledford2016-10-03 17:43:15 +0200
commit24f0c9c0ff0955c5849dfb40403a21c8c11b9d2a (patch)
treefa88fe71f5e431defb047c93b22dc3a5445f7345 /drivers/infiniband/hw/hns/hns_roce_cq.c
parentIB/hns: Validate mtu when modified qp (diff)
downloadkernel-qcow2-linux-24f0c9c0ff0955c5849dfb40403a21c8c11b9d2a.tar.gz
kernel-qcow2-linux-24f0c9c0ff0955c5849dfb40403a21c8c11b9d2a.tar.xz
kernel-qcow2-linux-24f0c9c0ff0955c5849dfb40403a21c8c11b9d2a.zip
IB/hns: Cq has not been freed
Cq has not been freed when fail to ib_copy_to_udata, so need to free it. Signed-off-by: Lijun Ou <oulijun@huawei.com> Signed-off-by: Peter Chen <luck.chen@huawei.com> Reviewed-by: Wei Hu (Xavier) <xavier.huwei@huawei.com> Signed-off-by: Salil Mehta <salil.mehta@huawei.com> Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/hw/hns/hns_roce_cq.c')
-rw-r--r--drivers/infiniband/hw/hns/hns_roce_cq.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/hns/hns_roce_cq.c b/drivers/infiniband/hw/hns/hns_roce_cq.c
index 3095f06099e6..097365932b09 100644
--- a/drivers/infiniband/hw/hns/hns_roce_cq.c
+++ b/drivers/infiniband/hw/hns/hns_roce_cq.c
@@ -357,12 +357,15 @@ struct ib_cq *hns_roce_ib_create_cq(struct ib_device *ib_dev,
if (context) {
if (ib_copy_to_udata(udata, &hr_cq->cqn, sizeof(u64))) {
ret = -EFAULT;
- goto err_mtt;
+ goto err_cqc;
}
}
return &hr_cq->ib_cq;
+err_cqc:
+ hns_roce_free_cq(hr_dev, hr_cq);
+
err_mtt:
hns_roce_mtt_cleanup(hr_dev, &hr_cq->hr_buf.hr_mtt);
if (context)