diff options
author | Jayamohan Kallickal <jayamohank@serverengines.com> | 2010-07-21 18:54:22 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:05:33 -0400 |
commit | 5db3f33d687c5a4ba589bf3af98c786399c6e213 (patch) | |
tree | ce92480d2ab9be266e0ac76513643e66d4cc39f0 | |
parent | 4f5af07e1bc4ae64b7a7ead5bf60f40a3115ceeb (diff) |
[SCSI] be2iscsi: Free tags allocated
This patch frees tags that are already allocated in case of
failure
Signed-off-by: Jayamohan Kallickal <jayamohank@serverengines.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
-rw-r--r-- | drivers/scsi/be2iscsi/be_mgmt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/scsi/be2iscsi/be_mgmt.c b/drivers/scsi/be2iscsi/be_mgmt.c index dcb70fac2fce..3036d9ee7902 100644 --- a/drivers/scsi/be2iscsi/be_mgmt.c +++ b/drivers/scsi/be2iscsi/be_mgmt.c | |||
@@ -168,6 +168,7 @@ unsigned int mgmt_invalidate_icds(struct beiscsi_hba *phba, | |||
168 | SE_DEBUG(DBG_LVL_1, | 168 | SE_DEBUG(DBG_LVL_1, |
169 | "Failed to alloc memory for mgmt_invalidate_icds\n"); | 169 | "Failed to alloc memory for mgmt_invalidate_icds\n"); |
170 | spin_unlock(&ctrl->mbox_lock); | 170 | spin_unlock(&ctrl->mbox_lock); |
171 | free_mcc_tag(&phba->ctrl, tag); | ||
171 | return 0; | 172 | return 0; |
172 | } | 173 | } |
173 | nonemb_cmd.size = sizeof(struct invalidate_commands_params_in); | 174 | nonemb_cmd.size = sizeof(struct invalidate_commands_params_in); |
@@ -330,6 +331,7 @@ int mgmt_open_connection(struct beiscsi_hba *phba, | |||
330 | shost_printk(KERN_ERR, phba->shost, "unknown addr family %d\n", | 331 | shost_printk(KERN_ERR, phba->shost, "unknown addr family %d\n", |
331 | dst_addr->sa_family); | 332 | dst_addr->sa_family); |
332 | spin_unlock(&ctrl->mbox_lock); | 333 | spin_unlock(&ctrl->mbox_lock); |
334 | free_mcc_tag(&phba->ctrl, tag); | ||
333 | return -EINVAL; | 335 | return -EINVAL; |
334 | 336 | ||
335 | } | 337 | } |