diff options
| author | Poul-Henning Kamp <phk@FreeBSD.org> | 2003-05-31 18:41:09 +0000 |
|---|---|---|
| committer | Poul-Henning Kamp <phk@FreeBSD.org> | 2003-05-31 18:41:09 +0000 |
| commit | 777d1b399038a27556121c5281f631b6a2b801a9 (patch) | |
| tree | d6f639e72db334d9e0955ebdcbe4f74749876374 /sys/dev/ciss | |
| parent | 0623afde6f31ef8cd1687b8009ed3b6c5dff0727 (diff) | |
| download | src-777d1b399038a27556121c5281f631b6a2b801a9.tar.gz src-777d1b399038a27556121c5281f631b6a2b801a9.zip | |
Fix off-by-one error in drive number check.
Don't return(foo(...)) in function returning void.
Found by: FlexeLint
Notes
Notes:
svn path=/head/; revision=115482
Diffstat (limited to 'sys/dev/ciss')
| -rw-r--r-- | sys/dev/ciss/ciss.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sys/dev/ciss/ciss.c b/sys/dev/ciss/ciss.c index 6009b442479d..9608a158e01a 100644 --- a/sys/dev/ciss/ciss.c +++ b/sys/dev/ciss/ciss.c @@ -973,7 +973,7 @@ ciss_init_logical(struct ciss_softc *sc) /* sanity-check reply */ ndrives = (ntohl(cll->list_size) / sizeof(union ciss_device_address)); - if ((ndrives < 0) || (ndrives > CISS_MAX_LOGICAL)) { + if ((ndrives < 0) || (ndrives >= CISS_MAX_LOGICAL)) { ciss_printf(sc, "adapter claims to report absurd number of logical drives (%d > %d)\n", ndrives, CISS_MAX_LOGICAL); return(ENXIO); @@ -2139,7 +2139,7 @@ ciss_cam_rescan_target(struct ciss_softc *sc, int target) static void ciss_cam_rescan_all(struct ciss_softc *sc) { - return(ciss_cam_rescan_target(sc, 0)); + ciss_cam_rescan_target(sc, 0); } static void @@ -2271,7 +2271,7 @@ ciss_cam_action_io(struct cam_sim *sim, struct ccb_scsiio *csio) /* check for I/O attempt to nonexistent device */ if ((bus != 0) || - (target > CISS_MAX_LOGICAL) || + (target >= CISS_MAX_LOGICAL) || (sc->ciss_logical[target].cl_status == CISS_LD_NONEXISTENT)) { debug(3, " device does not exist"); csio->ccb_h.status = CAM_REQ_CMP_ERR; |