aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/target
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2012-09-07 02:55:32 -0400
committerNicholas Bellinger <nab@linux-iscsi.org>2012-09-17 21:04:09 -0400
commit7875f179085608c659b676be74d33a2e515d8f43 (patch)
tree163cadd0a5f103a0adcc6d1dcdc13658a0c8480e /drivers/target
parentd81cb44726f050d7cf1be4afd9cb45d153b52066 (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.c3
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 */
328void ft_invl_hw_context(struct ft_cmd *cmd) 328void 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) {