scsi: bnx2fc: Remove meaningless bnx2fc_abts_cleanup() return value assignment
authorSeongJae Park <sjpark@amazon.de>
Fri, 18 Jun 2021 16:45:14 +0000 (16:45 +0000)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 23 Jun 2021 02:59:47 +0000 (22:59 -0400)
commit73b306a2bcb75e37b8065aa714ad2c6949c90ebf
treee5fa7cf851b1807b2a3a2b79364712fcb3df7d04
parentd94d8158e1841813624e9fecf93a12e64e004dd8
scsi: bnx2fc: Remove meaningless bnx2fc_abts_cleanup() return value assignment

Commit 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is already in
ABTS processing") made bnx2fc_eh_abort() return FAILED when io_req was
alrady in ABTS processing, regardless of the return value of
bnx2fc_abts_cleanup().  However, the change left the assignment of the
return value of bnx2fc_abts_cleanup(). Remove this.

This issue was discovered and resolved using Coverity Static Analysis
Security Testing (SAST) by Synopsys, Inc.

Link: https://lore.kernel.org/r/20210618164514.6299-1-sj38.park@gmail.com
Fixes: 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is already in ABTS processing")
Acked-by: Saurav Kashyap <skashyap@marvell.com>
Signed-off-by: SeongJae Park <sjpark@amazon.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/bnx2fc/bnx2fc_io.c