diff options
author | Jörn Engel <joern@logfs.org> | 2012-03-15 15:06:58 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-03-15 22:16:09 -0400 |
commit | 281689051a628e5341ce1efcfafde9d60f2f6fbb (patch) | |
tree | 15809410fe5d1c8540622f5bd7c726517b22ddd3 /drivers/target/tcm_fc | |
parent | f2083241f23722207676025abbb45a301d412e69 (diff) |
target: remove obvious warnings
Get rid of a bunch of write-only variables. In a number of cases I
suspect actual bugs to be present, so I left all of those for a second
look.
(nab: fix lio-core patch fuzz)
Signed-off-by: Joern Engel <joern@logfs.org>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/tcm_fc')
-rw-r--r-- | drivers/target/tcm_fc/tfc_sess.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/target/tcm_fc/tfc_sess.c b/drivers/target/tcm_fc/tfc_sess.c index a1d1f3c86447..b30f92bdc1a4 100644 --- a/drivers/target/tcm_fc/tfc_sess.c +++ b/drivers/target/tcm_fc/tfc_sess.c | |||
@@ -319,11 +319,9 @@ int ft_sess_shutdown(struct se_session *se_sess) | |||
319 | void ft_sess_close(struct se_session *se_sess) | 319 | void ft_sess_close(struct se_session *se_sess) |
320 | { | 320 | { |
321 | struct ft_sess *sess = se_sess->fabric_sess_ptr; | 321 | struct ft_sess *sess = se_sess->fabric_sess_ptr; |
322 | struct fc_lport *lport; | ||
323 | u32 port_id; | 322 | u32 port_id; |
324 | 323 | ||
325 | mutex_lock(&ft_lport_lock); | 324 | mutex_lock(&ft_lport_lock); |
326 | lport = sess->tport->lport; | ||
327 | port_id = sess->port_id; | 325 | port_id = sess->port_id; |
328 | if (port_id == -1) { | 326 | if (port_id == -1) { |
329 | mutex_unlock(&ft_lport_lock); | 327 | mutex_unlock(&ft_lport_lock); |