]> asedeno.scripts.mit.edu Git - linux.git/commit
scsi: hisi_sas: check sas_dev gone earlier in hisi_sas_abort_task()
authorXiang Chen <chenxiang66@hisilicon.com>
Wed, 2 May 2018 15:56:28 +0000 (23:56 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 8 May 2018 05:10:44 +0000 (01:10 -0400)
commitc6ef895472696cac8e50f0ce69b301cc10233a67
treeef923fcef58f4f466573f6cb19ed9d5584b57138
parenta14da7a20d499c54ebe53e430283e3bc17d3e89f
scsi: hisi_sas: check sas_dev gone earlier in hisi_sas_abort_task()

It is possible to dereference a NULL-pointer in hisi_sas_abort_task() in
special scenario when the device has been removed.

If an SMP task times-out, it will call hisi_sas_abort_task() to
recover. And currently there is a check in hisi_sas_abort_task() to
avoid the situation of processing the abort for the removed device.

However we have an ordering problem, in that we may reference a task for
the removed device before checking if the device has been removed.

Fix this by only referencing the sas_dev after we know it is still
present.

Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/hisi_sas/hisi_sas_main.c