projects / linux.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: d251193)
scsi: target: Remove redundant assignment to variable 'ret'
authorJing Xiangfeng <jingxiangfeng@huawei.com>
Mon, 14 Sep 2020 02:32:07 +0000 (10:32 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 15 Sep 2020 22:06:00 +0000 (18:06 -0400)
The variable ret has been initialized with a value '0'. The assignment in
switch-case is redundant. Remove it.

Link: https://lore.kernel.org/r/20200914023207.113792-1-jingxiangfeng@huawei.com
Reviewed-by: Maurizio Lombardi <mlombard@redhat.com>
Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/iscsi/iscsi_target.c patch | blob | history

diff --git a/drivers/target/iscsi/iscsi_target.c b/drivers/target/iscsi/iscsi_target.c
index 2ec778e..833f043 100644 (file)
--- a/drivers/target/iscsi/iscsi_target.c
+++ b/drivers/target/iscsi/iscsi_target.c
@@ -4529,7 +4529,6 @@ int iscsit_logout_post_handler(
                        iscsit_logout_post_handler_closesession(conn);
                        break;
                }
-               ret = 0;
                break;
        case ISCSI_LOGOUT_REASON_CLOSE_CONNECTION:
                if (conn->cid == cmd->logout_cid) {
@@ -4540,7 +4539,6 @@ int iscsit_logout_post_handler(
                                iscsit_logout_post_handler_samecid(conn);
                                break;
                        }
-                       ret = 0;
                } else {
                        switch (cmd->logout_response) {
                        case ISCSI_LOGOUT_SUCCESS: