Commit a5c277ec authored by Maurizio Lombardi's avatar Maurizio Lombardi Committed by Christoph Hellwig

qla4xxx: check the return value of dma_alloc_coherent()

the qla4xxx_alloc_fw_dump() calls dma_alloc_coherent() but does not
check its return value.
Signed-off-by: default avatarMaurizio Lombardi <mlombard@redhat.com>
Acked-By: default avatarNilesh Javali <nilesh.javali@qlogic.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent 4d0a21f1
...@@ -334,6 +334,12 @@ void qla4xxx_alloc_fw_dump(struct scsi_qla_host *ha) ...@@ -334,6 +334,12 @@ void qla4xxx_alloc_fw_dump(struct scsi_qla_host *ha)
/* Allocate memory for saving the template */ /* Allocate memory for saving the template */
md_tmp = dma_alloc_coherent(&ha->pdev->dev, ha->fw_dump_tmplt_size, md_tmp = dma_alloc_coherent(&ha->pdev->dev, ha->fw_dump_tmplt_size,
&md_tmp_dma, GFP_KERNEL); &md_tmp_dma, GFP_KERNEL);
if (!md_tmp) {
ql4_printk(KERN_INFO, ha,
"scsi%ld: Failed to allocate DMA memory\n",
ha->host_no);
return;
}
/* Request template */ /* Request template */
status = qla4xxx_get_minidump_template(ha, md_tmp_dma); status = qla4xxx_get_minidump_template(ha, md_tmp_dma);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment