2021-06-17 13:40:18

by Colin King

[permalink] [raw]
Subject: [PATCH] scsi: target/iscsi: remove redundant continue statement

From: Colin Ian King <[email protected]>

The continue statement at the end of a loop has no effect,
remove it.

Addresses-Coverity: ("Continue has no effect")
Signed-off-by: Colin Ian King <[email protected]>
---
drivers/target/iscsi/iscsi_target_erl1.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/target/iscsi/iscsi_target_erl1.c b/drivers/target/iscsi/iscsi_target_erl1.c
index cd670cb9b8fb..0dd52f484fec 100644
--- a/drivers/target/iscsi/iscsi_target_erl1.c
+++ b/drivers/target/iscsi/iscsi_target_erl1.c
@@ -871,8 +871,6 @@ int iscsit_execute_ooo_cmdsns(struct iscsi_session *sess)

if (iscsit_execute_cmd(cmd, 1) < 0)
return -1;
-
- continue;
}

return ooo_count;
--
2.31.1


2021-06-19 09:02:39

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH] scsi: target/iscsi: remove redundant continue statement


Colin,

> The continue statement at the end of a loop has no effect, remove it.

Applied to 5.14/scsi-staging, thanks!

--
Martin K. Petersen Oracle Linux Engineering