]> git.baikalelectronics.ru Git - kernel.git/commit
scsi: qla2xxx: Fix NULL pointer crash due to probe failure
authorhimanshu.madhani@cavium.com <himanshu.madhani@cavium.com>
Tue, 16 Jan 2018 04:46:46 +0000 (20:46 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 17 Jan 2018 06:34:24 +0000 (01:34 -0500)
commit7803989e0050d2062bb77f693f147f3a5a7f90a0
tree22454a232bd4064f661442894ff54b58f6be2ea8
parent3634437948048268dd4dc52290bc4b95af4a80e7
scsi: qla2xxx: Fix NULL pointer crash due to probe failure

This patch fixes regression added by commit 60446cc8b45ad
("scsi: qla2xxx: Add multiple queue pair functionality.").

When driver is not able to get reqeusted IRQs from the system, driver will
attempt tp clean up memory before failing hardware probe. During this cleanup,
driver assigns NULL value to the pointer which has not been allocated by
driver yet. This results in a NULL pointer access.

Log file will show following message and stack trace

qla2xxx [0000:a3:00.1]-00c7:21: MSI-X: Failed to enable support, giving up -- 32/-1.
qla2xxx [0000:a3:00.1]-0037:21: Falling back-to MSI mode --1.
qla2xxx [0000:a3:00.1]-003a:21: Failed to reserve interrupt 821 already in use.
BUG: unable to handle kernel NULL pointer dereference at (null)
IP: [<ffffffffc010c4b6>] qla2x00_probe_one+0x18b6/0x2730 [qla2xxx]
PGD 0
Oops: 0002 [#1] SMP

Fixes: 60446cc8b45ad ("scsi: qla2xxx: Add multiple queue pair functionality.").
Cc: <stable@vger.kernel.org> # 4.10
Signed-off-by: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_os.c