[SCSI] Return -EINVAL when "id == max_id" in scsi_scan_host_selected()
authorAmit Arora <aarora@in.ibm.com>
Fri, 19 May 2006 23:14:50 +0000 (16:14 -0700)
committerJames Bottomley <jejb@mulgrave.il.steeleye.com>
Sun, 28 May 2006 17:01:23 +0000 (13:01 -0400)
The scsi_scan_host_selected() should return -EINVAL when the id is equal
to the max_id. Currently it uses ">" when comparing with max_id, and
hence leaves the border case when "id==max_id".
The channel and lun have values valid from 0 up to,
and including, max_channel or max_lun. But, the valid values for id
range from 0 to max_id-1. This patch fixes the problem.

Signed-off-by: Amit Arora <aarora@in.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
drivers/scsi/scsi_scan.c

index f85d9100f554ee7a232e177074a5c6ccf38a1046..fd97d07577ad042f328a7b30b103a0abb6a9b405 100644 (file)
@@ -1503,7 +1503,7 @@ int scsi_scan_host_selected(struct Scsi_Host *shost, unsigned int channel,
                __FUNCTION__, channel, id, lun));
 
        if (((channel != SCAN_WILD_CARD) && (channel > shost->max_channel)) ||
-           ((id != SCAN_WILD_CARD) && (id > shost->max_id)) ||
+           ((id != SCAN_WILD_CARD) && (id >= shost->max_id)) ||
            ((lun != SCAN_WILD_CARD) && (lun > shost->max_lun)))
                return -EINVAL;