diff options
author | 2019-08-08 20:01:37 -0700 | |
---|---|---|
committer | 2019-08-12 21:34:05 -0400 | |
commit | 53be100b36aeca6bbc611edd07ce65fdba8b18ac (patch) | |
tree | 14077ccf6f6adc313c606b6950d9264aab57bb24 /drivers/scsi/qla2xxx/qla_nvme.c | |
parent | scsi: qla2xxx: Simplify qlt_lport_dump() (diff) | |
download | linux-53be100b36aeca6bbc611edd07ce65fdba8b18ac.tar.gz linux-53be100b36aeca6bbc611edd07ce65fdba8b18ac.tar.bz2 linux-53be100b36aeca6bbc611edd07ce65fdba8b18ac.zip |
scsi: qla2xxx: Remove a superfluous pointer check
Checking a pointer after it has been dereferenced is not useful. This was
detected by Coverity.
Cc: Himanshu Madhani <hmadhani@marvell.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Tested-by: Himanshu Madhani <hmadhani@marvell.com>
Reviewed-by: Himanshu Madhani <hmadhani@marvell.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_nvme.c')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_nvme.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_nvme.c b/drivers/scsi/qla2xxx/qla_nvme.c index 16028ee8c7a7..bba25c38a118 100644 --- a/drivers/scsi/qla2xxx/qla_nvme.c +++ b/drivers/scsi/qla2xxx/qla_nvme.c @@ -222,7 +222,7 @@ static void qla_nvme_abort_work(struct work_struct *work) "%s called for sp=%p, hndl=%x on fcport=%p deleted=%d\n", __func__, sp, sp->handle, fcport, fcport->deleted); - if (!ha->flags.fw_started && (fcport && fcport->deleted)) + if (!ha->flags.fw_started && fcport->deleted) goto out; if (ha->flags.host_shutting_down) { |