diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-09-07 02:55:32 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-09-17 21:04:09 -0400 |
commit | 7875f179085608c659b676be74d33a2e515d8f43 (patch) | |
tree | 163cadd0a5f103a0adcc6d1dcdc13658a0c8480e /drivers/target | |
parent | d81cb44726f050d7cf1be4afd9cb45d153b52066 (diff) |
tcm_fc: move the dereference below the NULL test
The dereference should be moved below the NULL test.
spatch with a semantic match is used to found this.
(http://coccinelle.lip6.fr/)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
-rw-r--r-- | drivers/target/tcm_fc/tfc_io.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/target/tcm_fc/tfc_io.c b/drivers/target/tcm_fc/tfc_io.c index d051102679a4..b6fd4cf42840 100644 --- a/drivers/target/tcm_fc/tfc_io.c +++ b/drivers/target/tcm_fc/tfc_io.c | |||
@@ -327,11 +327,12 @@ drop: | |||
327 | */ | 327 | */ |
328 | void ft_invl_hw_context(struct ft_cmd *cmd) | 328 | void ft_invl_hw_context(struct ft_cmd *cmd) |
329 | { | 329 | { |
330 | struct fc_seq *seq = cmd->seq; | 330 | struct fc_seq *seq; |
331 | struct fc_exch *ep = NULL; | 331 | struct fc_exch *ep = NULL; |
332 | struct fc_lport *lport = NULL; | 332 | struct fc_lport *lport = NULL; |
333 | 333 | ||
334 | BUG_ON(!cmd); | 334 | BUG_ON(!cmd); |
335 | seq = cmd->seq; | ||
335 | 336 | ||
336 | /* Cleanup the DDP context in HW if DDP was setup */ | 337 | /* Cleanup the DDP context in HW if DDP was setup */ |
337 | if (cmd->was_ddp_setup && seq) { | 338 | if (cmd->was_ddp_setup && seq) { |