aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGlenn Streiff <gstreiff@neteffect.com>2008-02-21 09:17:54 -0500
committerRoland Dreier <rolandd@cisco.com>2008-02-26 19:24:29 -0500
commitf84fba6f969065c6622669bbaa955c26fc1461ae (patch)
treeb562989918c69b271c04b83df7b27df786229d8a
parenta4435febd4c0f14b25159dca249ecf91301c7c76 (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>
-rw-r--r--drivers/infiniband/hw/nes/nes_verbs.c3
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);