| From: Dan Carpenter <dan.carpenter@oracle.com> |
| Date: Tue, 3 Dec 2019 12:45:09 +0300 |
| Subject: scsi: iscsi: qla4xxx: fix double free in probe |
| |
| commit fee92f25777789d73e1936b91472e9c4644457c8 upstream. |
| |
| On this error path we call qla4xxx_mem_free() and then the caller also |
| calls qla4xxx_free_adapter() which calls qla4xxx_mem_free(). It leads to a |
| couple double frees: |
| |
| drivers/scsi/qla4xxx/ql4_os.c:8856 qla4xxx_probe_adapter() warn: 'ha->chap_dma_pool' double freed |
| drivers/scsi/qla4xxx/ql4_os.c:8856 qla4xxx_probe_adapter() warn: 'ha->fw_ddb_dma_pool' double freed |
| |
| Fixes: afaf5a2d341d ("[SCSI] Initial Commit of qla4xxx") |
| Link: https://lore.kernel.org/r/20191203094421.hw7ex7qr3j2rbsmx@kili.mountain |
| Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> |
| Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> |
| Signed-off-by: Ben Hutchings <ben@decadent.org.uk> |
| --- |
| drivers/scsi/qla4xxx/ql4_os.c | 1 - |
| 1 file changed, 1 deletion(-) |
| |
| --- a/drivers/scsi/qla4xxx/ql4_os.c |
| +++ b/drivers/scsi/qla4xxx/ql4_os.c |
| @@ -4269,7 +4269,6 @@ static int qla4xxx_mem_alloc(struct scsi |
| return QLA_SUCCESS; |
| |
| mem_alloc_error_exit: |
| - qla4xxx_mem_free(ha); |
| return QLA_ERROR; |
| } |
| |