Skip to content

Commit 8ca5d26

Browse files
ColinIanKingsmfrench
authored andcommitted
cifs: remove redundant variable tcon_exist
The variable tcon_exist is being assigned however it is never read, the variable is redundant and can be removed. Cleans up clang scan build warning: warning: Although the value stored to 'tcon_exist' is used in the enclosing expression, the value is never actually readfrom 'tcon_exist' [deadcode.DeadStores] Signed-off-by: Colin Ian King <[email protected]> Signed-off-by: Steve French <[email protected]>
1 parent 84e9a2d commit 8ca5d26

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

fs/smb/client/smb2pdu.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3918,7 +3918,7 @@ void smb2_reconnect_server(struct work_struct *work)
39183918
struct cifs_ses *ses, *ses2;
39193919
struct cifs_tcon *tcon, *tcon2;
39203920
struct list_head tmp_list, tmp_ses_list;
3921-
bool tcon_exist = false, ses_exist = false;
3921+
bool ses_exist = false;
39223922
bool tcon_selected = false;
39233923
int rc;
39243924
bool resched = false;
@@ -3964,7 +3964,7 @@ void smb2_reconnect_server(struct work_struct *work)
39643964
if (tcon->need_reconnect || tcon->need_reopen_files) {
39653965
tcon->tc_count++;
39663966
list_add_tail(&tcon->rlist, &tmp_list);
3967-
tcon_selected = tcon_exist = true;
3967+
tcon_selected = true;
39683968
}
39693969
}
39703970
/*
@@ -3973,7 +3973,7 @@ void smb2_reconnect_server(struct work_struct *work)
39733973
*/
39743974
if (ses->tcon_ipc && ses->tcon_ipc->need_reconnect) {
39753975
list_add_tail(&ses->tcon_ipc->rlist, &tmp_list);
3976-
tcon_selected = tcon_exist = true;
3976+
tcon_selected = true;
39773977
cifs_smb_ses_inc_refcount(ses);
39783978
}
39793979
/*

0 commit comments

Comments
 (0)