diff options
author | Hannes Reinecke <hare@suse.de> | 2006-05-23 04:29:28 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-05-28 13:07:41 -0400 |
commit | 2b89dad0c7e3b03d45d9674ee9a7b49670df098e (patch) | |
tree | 8f480172c865e9d6cfab9ed4888a043bc75ab4ce /drivers/scsi/aic7xxx | |
parent | 091686d3b5569d2ec76b9e6dab7f3608b6f64497 (diff) |
[SCSI] audit drivers for incorrect max_id use
max_id now means the maximum number of ids on the bus, which means it
is one greater than the largest possible id number.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/aic7xxx')
-rw-r--r-- | drivers/scsi/aic7xxx/aic79xx_core.c | 4 | ||||
-rw-r--r-- | drivers/scsi/aic7xxx/aic7xxx_core.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/drivers/scsi/aic7xxx/aic79xx_core.c b/drivers/scsi/aic7xxx/aic79xx_core.c index 08771f6f6859..e14244aa69d7 100644 --- a/drivers/scsi/aic7xxx/aic79xx_core.c +++ b/drivers/scsi/aic7xxx/aic79xx_core.c | |||
@@ -9396,8 +9396,8 @@ ahd_find_tmode_devs(struct ahd_softc *ahd, struct cam_sim *sim, union ccb *ccb, | |||
9396 | } else { | 9396 | } else { |
9397 | u_int max_id; | 9397 | u_int max_id; |
9398 | 9398 | ||
9399 | max_id = (ahd->features & AHD_WIDE) ? 15 : 7; | 9399 | max_id = (ahd->features & AHD_WIDE) ? 16 : 8; |
9400 | if (ccb->ccb_h.target_id > max_id) | 9400 | if (ccb->ccb_h.target_id >= max_id) |
9401 | return (CAM_TID_INVALID); | 9401 | return (CAM_TID_INVALID); |
9402 | 9402 | ||
9403 | if (ccb->ccb_h.target_lun >= AHD_NUM_LUNS) | 9403 | if (ccb->ccb_h.target_lun >= AHD_NUM_LUNS) |
diff --git a/drivers/scsi/aic7xxx/aic7xxx_core.c b/drivers/scsi/aic7xxx/aic7xxx_core.c index d37566978fba..50a3dd047cfe 100644 --- a/drivers/scsi/aic7xxx/aic7xxx_core.c +++ b/drivers/scsi/aic7xxx/aic7xxx_core.c | |||
@@ -6774,8 +6774,8 @@ ahc_find_tmode_devs(struct ahc_softc *ahc, struct cam_sim *sim, union ccb *ccb, | |||
6774 | } else { | 6774 | } else { |
6775 | u_int max_id; | 6775 | u_int max_id; |
6776 | 6776 | ||
6777 | max_id = (ahc->features & AHC_WIDE) ? 15 : 7; | 6777 | max_id = (ahc->features & AHC_WIDE) ? 16 : 8; |
6778 | if (ccb->ccb_h.target_id > max_id) | 6778 | if (ccb->ccb_h.target_id >= max_id) |
6779 | return (CAM_TID_INVALID); | 6779 | return (CAM_TID_INVALID); |
6780 | 6780 | ||
6781 | if (ccb->ccb_h.target_lun >= AHC_NUM_LUNS) | 6781 | if (ccb->ccb_h.target_lun >= AHC_NUM_LUNS) |