diff options
| author | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-11-12 12:29:51 -0500 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-12-06 01:00:51 -0500 |
| commit | 6297b07cbc42eb6b53eb88fce70a4727ea230797 (patch) | |
| tree | 34996a19baaa466e274b361e4437da97b85d249f /drivers/target/tcm_fc | |
| parent | 97c34f3b04978799fd6eada69e1a8d84b74d9599 (diff) | |
target: Drop config_item_name usage in fabric TFO->free_wwn()
This patch removes config_item_name() informational usage of
TFO->free_wwn() treewide in loopback, tcm_fc, ib_srpt and
tcm_vhost module code.
Using v4 target_core_fabric_configfs.c logic, a fabric call for
config_item_name() in TFO->drop_wwn() context returns NULL as
target_fabric_drop_wwn() invoking config_item_put() ->
config_group_put() will release fabric_port->port_wwn.wwn_group
before the last config_item_put() -> TFO->drop_wwn() is
invoked.
Reported-by: Bart Van Assche <bvanassche@acm.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_conf.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/target/tcm_fc/tfc_conf.c b/drivers/target/tcm_fc/tfc_conf.c index 5f770412ca40..9402b7387cac 100644 --- a/drivers/target/tcm_fc/tfc_conf.c +++ b/drivers/target/tcm_fc/tfc_conf.c | |||
| @@ -436,8 +436,7 @@ static void ft_del_lport(struct se_wwn *wwn) | |||
| 436 | struct ft_lport_acl *lacl = container_of(wwn, | 436 | struct ft_lport_acl *lacl = container_of(wwn, |
| 437 | struct ft_lport_acl, fc_lport_wwn); | 437 | struct ft_lport_acl, fc_lport_wwn); |
| 438 | 438 | ||
| 439 | pr_debug("del lport %s\n", | 439 | pr_debug("del lport %s\n", lacl->name); |
| 440 | config_item_name(&wwn->wwn_group.cg_item)); | ||
| 441 | mutex_lock(&ft_lport_lock); | 440 | mutex_lock(&ft_lport_lock); |
| 442 | list_del(&lacl->list); | 441 | list_del(&lacl->list); |
| 443 | mutex_unlock(&ft_lport_lock); | 442 | mutex_unlock(&ft_lport_lock); |
