From: Fubo Chen Date: Wed, 9 Feb 2011 23:34:48 +0000 (-0800) Subject: [SCSI] target: fixed missing lock drop in error path X-Git-Tag: upstream/snapshot3+hdmi~11443^2~8 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=85dc98d93f3dc41cce54118a7abab9e6aa616dd2;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git [SCSI] target: fixed missing lock drop in error path The struct se_node_acl->device_list_lock needs to be released if either sanity check for struct se_dev_entry->se_lun_acl or deve->se_lun fails. Signed-off-by: Fubo Chen Signed-off-by: Nicholas A. Bellinger Signed-off-by: James Bottomley --- diff --git a/drivers/target/target_core_device.c b/drivers/target/target_core_device.c index 969d727..9551ab5 100644 --- a/drivers/target/target_core_device.c +++ b/drivers/target/target_core_device.c @@ -395,12 +395,14 @@ int core_update_device_list_for_node( printk(KERN_ERR "struct se_dev_entry->se_lun_acl" " already set for demo mode -> explict" " LUN ACL transition\n"); + spin_unlock_irq(&nacl->device_list_lock); return -1; } if (deve->se_lun != lun) { printk(KERN_ERR "struct se_dev_entry->se_lun does" " match passed struct se_lun for demo mode" " -> explict LUN ACL transition\n"); + spin_unlock_irq(&nacl->device_list_lock); return -1; } deve->se_lun_acl = lun_acl;