aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorJames Bottomley <jejb@mulgrave.(none)>2005-10-28 14:17:30 -0400
committerJames Bottomley <jejb@mulgrave.(none)>2005-10-28 14:17:30 -0400
commit7a691bd34130920bef4d118a3f555ebc48544a63 (patch)
tree50b675e97e1525d2e07bf3c5bdc0098623768694 /drivers
parentd578a4258bf674ef86b16bf29267fb302125aaee (diff)
[SCSI] avoid overflows in disk size calculations
Be more careful about doing the arithmetic in the non-LBD case. Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/scsi/sd.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c
index 9a1dc0cea03c..98da5c8057bf 100644
--- a/drivers/scsi/sd.c
+++ b/drivers/scsi/sd.c
@@ -1253,14 +1253,13 @@ got_data:
1253 * Jacques Gelinas (Jacques@solucorp.qc.ca) 1253 * Jacques Gelinas (Jacques@solucorp.qc.ca)
1254 */ 1254 */
1255 int hard_sector = sector_size; 1255 int hard_sector = sector_size;
1256 sector_t sz = sdkp->capacity * (hard_sector/256); 1256 sector_t sz = (sdkp->capacity/2) * (hard_sector/256);
1257 request_queue_t *queue = sdp->request_queue; 1257 request_queue_t *queue = sdp->request_queue;
1258 sector_t mb; 1258 sector_t mb = sz;
1259 1259
1260 blk_queue_hardsect_size(queue, hard_sector); 1260 blk_queue_hardsect_size(queue, hard_sector);
1261 /* avoid 64-bit division on 32-bit platforms */ 1261 /* avoid 64-bit division on 32-bit platforms */
1262 mb = sz >> 1; 1262 sector_div(sz, 625);
1263 sector_div(sz, 1250);
1264 mb -= sz - 974; 1263 mb -= sz - 974;
1265 sector_div(mb, 1950); 1264 sector_div(mb, 1950);
1266 1265