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>
This commit is contained in:
parent
a4435febd4
commit
f84fba6f96
|
@ -1832,9 +1832,6 @@ static struct ib_cq *nes_create_cq(struct ib_device *ibdev, int entries,
|
||||||
spin_unlock_irqrestore(&nesdev->cqp.lock, flags);
|
spin_unlock_irqrestore(&nesdev->cqp.lock, flags);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
nes_debug(NES_DBG_CQ, "iWARP CQ%u create timeout expired, major code = 0x%04X,"
|
|
||||||
" minor code = 0x%04X\n",
|
|
||||||
nescq->hw_cq.cq_number, cqp_request->major_code, cqp_request->minor_code);
|
|
||||||
if (!context)
|
if (!context)
|
||||||
pci_free_consistent(nesdev->pcidev, nescq->cq_mem_size, mem,
|
pci_free_consistent(nesdev->pcidev, nescq->cq_mem_size, mem,
|
||||||
nescq->hw_cq.cq_pbase);
|
nescq->hw_cq.cq_pbase);
|
||||||
|
|
Loading…
Reference in a new issue