| Message ID | 20210618164514.6299-1-sj38.park@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | scsi: bnx2fc: Remove meaningless 'bnx2fc_abts_cleanup()' return value assignment | expand |
Hi SeongJae, > -----Original Message----- > From: SeongJae Park <sj38.park@gmail.com> > Sent: Friday, June 18, 2021 10:15 PM > To: Saurav Kashyap <skashyap@marvell.com> > Cc: Javed Hasan <jhasan@marvell.com>; GR-QLogic-Storage-Upstream <GR- > QLogic-Storage-Upstream@marvell.com>; jejb@linux.ibm.com; > martin.petersen@oracle.com; himanshu.madhani@oracle.com; linux- > scsi@vger.kernel.org; linux-kernel@vger.kernel.org; SeongJae Park > <sjpark@amazon.de> > Subject: [EXT] [PATCH] scsi: bnx2fc: Remove meaningless > 'bnx2fc_abts_cleanup()' return value assignment > > External Email > > ---------------------------------------------------------------------- > From: SeongJae Park <sjpark@amazon.de> > > Commit 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is already > in ABTS processing") made 'bnx2fc_eh_abort()' to return 'FAILED' > when 'io_req' is alrady in ABTS processing, regardless of the return > value of 'bnx2fc_abts_cleanup()'. But, it left the assignment of the > return value of 'bnx2fc_abts_cleanup()' to 'rc', which is meaningless > now. This commit removes it. > > This issue was discovered and resolved using Coverity Static Analysis > Security Testing (SAST) by Synopsys, Inc. > > Fixes: 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is already in ABTS > processing") > Signed-off-by: SeongJae Park <sjpark@amazon.de> > --- > drivers/scsi/bnx2fc/bnx2fc_io.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/bnx2fc/bnx2fc_io.c b/drivers/scsi/bnx2fc/bnx2fc_io.c > index ed300a279a38..f2996a9b2f63 100644 > --- a/drivers/scsi/bnx2fc/bnx2fc_io.c > +++ b/drivers/scsi/bnx2fc/bnx2fc_io.c > @@ -1213,7 +1213,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd) > * cleanup the command and return that I/O was successfully > * aborted. > */ > - rc = bnx2fc_abts_cleanup(io_req); > + bnx2fc_abts_cleanup(io_req); > /* This only occurs when an task abort was requested while > ABTS > is in progress. Setting the IO_CLEANUP flag will skip the > RRQ process in the case when the fw generated SCSI_CMD > cmpl Acked-by: Saurav Kashyap <skashyap@marvell.com> Thanks, ~Saurav > -- > 2.17.1
SeongJae, > Commit 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is > already in ABTS processing") made 'bnx2fc_eh_abort()' to return > 'FAILED' when 'io_req' is alrady in ABTS processing, regardless of the > return value of 'bnx2fc_abts_cleanup()'. But, it left the assignment > of the return value of 'bnx2fc_abts_cleanup()' to 'rc', which is > meaningless now. This commit removes it. Applied to 5.14/scsi-staging, thanks!
On Fri, 18 Jun 2021 16:45:14 +0000, SeongJae Park wrote: > Commit 122c81c563b0 ("scsi: bnx2fc: Return failure if io_req is already > in ABTS processing") made 'bnx2fc_eh_abort()' to return 'FAILED' > when 'io_req' is alrady in ABTS processing, regardless of the return > value of 'bnx2fc_abts_cleanup()'. But, it left the assignment of the > return value of 'bnx2fc_abts_cleanup()' to 'rc', which is meaningless > now. This commit removes it. > > [...] Applied to 5.14/scsi-queue, thanks! [1/1] scsi: bnx2fc: Remove meaningless 'bnx2fc_abts_cleanup()' return value assignment https://git.kernel.org/mkp/scsi/c/73b306a2bcb7
diff --git a/drivers/scsi/bnx2fc/bnx2fc_io.c b/drivers/scsi/bnx2fc/bnx2fc_io.c index ed300a279a38..f2996a9b2f63 100644 --- a/drivers/scsi/bnx2fc/bnx2fc_io.c +++ b/drivers/scsi/bnx2fc/bnx2fc_io.c @@ -1213,7 +1213,7 @@ int bnx2fc_eh_abort(struct scsi_cmnd *sc_cmd) * cleanup the command and return that I/O was successfully * aborted. */ - rc = bnx2fc_abts_cleanup(io_req); + bnx2fc_abts_cleanup(io_req); /* This only occurs when an task abort was requested while ABTS is in progress. Setting the IO_CLEANUP flag will skip the RRQ process in the case when the fw generated SCSI_CMD cmpl