diff options
author | Glenn Streiff <gstreiff@neteffect.com> | 2008-02-21 09:17:54 -0500 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2008-02-26 19:24:29 -0500 |
commit | f84fba6f969065c6622669bbaa955c26fc1461ae (patch) | |
tree | b562989918c69b271c04b83df7b27df786229d8a /drivers/infiniband/hw/nes | |
parent | a4435febd4c0f14b25159dca249ecf91301c7c76 (diff) |
RDMA/nes: Fix use-after-free in nes_create_cq()
Just delete the debugging statement so we don't use cqp_request after
freeing it. Adrian Bunk flagged this use-after-free issue spotted by
the Coverity checker.
Signed-off-by: Glenn Streiff <gstreiff@neteffect.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers/infiniband/hw/nes')
-rw-r--r-- | drivers/infiniband/hw/nes/nes_verbs.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/infiniband/hw/nes/nes_verbs.c b/drivers/infiniband/hw/nes/nes_verbs.c index 692f0d821301..a651e9d9f0ef 100644 --- a/drivers/infiniband/hw/nes/nes_verbs.c +++ b/drivers/infiniband/hw/nes/nes_verbs.c | |||
@@ -1832,9 +1832,6 @@ static struct ib_cq *nes_create_cq(struct ib_device *ibdev, int entries, | |||
1832 | spin_unlock_irqrestore(&nesdev->cqp.lock, flags); | 1832 | spin_unlock_irqrestore(&nesdev->cqp.lock, flags); |
1833 | } | 1833 | } |
1834 | } | 1834 | } |
1835 | nes_debug(NES_DBG_CQ, "iWARP CQ%u create timeout expired, major code = 0x%04X," | ||
1836 | " minor code = 0x%04X\n", | ||
1837 | nescq->hw_cq.cq_number, cqp_request->major_code, cqp_request->minor_code); | ||
1838 | if (!context) | 1835 | if (!context) |
1839 | pci_free_consistent(nesdev->pcidev, nescq->cq_mem_size, mem, | 1836 | pci_free_consistent(nesdev->pcidev, nescq->cq_mem_size, mem, |
1840 | nescq->hw_cq.cq_pbase); | 1837 | nescq->hw_cq.cq_pbase); |