target: Check LUN numbers in transport_get_lun_for_[cmd,tmr]
authorFubo Chen <fubo.chen@gmail.com>
Sun, 13 Feb 2011 23:13:42 +0000 (15:13 -0800)
committerNicholas Bellinger <nab@linux-iscsi.org>
Fri, 22 Jul 2011 09:37:42 +0000 (09:37 +0000)
This patch checks the passed 'unpacked_lun' against TRANSPORT_MAX_LUNS_PER_TPG
before reading from struct se_node_acl->device_list[].

Signed-off-by: Fubo Chen <fubo.chen@gmail.com>
Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_device.c

index ba698ea..f5516a1 100644 (file)
@@ -65,6 +65,12 @@ int transport_get_lun_for_cmd(
        unsigned long flags;
        int read_only = 0;
 
+       if (unpacked_lun >= TRANSPORT_MAX_LUNS_PER_TPG) {
+               se_cmd->scsi_sense_reason = TCM_NON_EXISTENT_LUN;
+               se_cmd->se_cmd_flags |= SCF_SCSI_CDB_EXCEPTION;
+               return -1;
+       }
+
        spin_lock_irq(&SE_NODE_ACL(se_sess)->device_list_lock);
        deve = se_cmd->se_deve =
                        &SE_NODE_ACL(se_sess)->device_list[unpacked_lun];
@@ -187,6 +193,12 @@ int transport_get_lun_for_tmr(
        struct se_session *se_sess = SE_SESS(se_cmd);
        struct se_tmr_req *se_tmr = se_cmd->se_tmr_req;
 
+       if (unpacked_lun >= TRANSPORT_MAX_LUNS_PER_TPG) {
+               se_cmd->scsi_sense_reason = TCM_NON_EXISTENT_LUN;
+               se_cmd->se_cmd_flags |= SCF_SCSI_CDB_EXCEPTION;
+               return -1;
+       }
+
        spin_lock_irq(&SE_NODE_ACL(se_sess)->device_list_lock);
        deve = se_cmd->se_deve =
                        &SE_NODE_ACL(se_sess)->device_list[unpacked_lun];