diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-31 14:15:25 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2009-03-31 14:15:25 -0400 |
commit | 41fa9f863baacd32dd049daf8050d55a0c9e6f1a (patch) | |
tree | 44cc612753746cd459e122f102c72f33b36aa6b7 /drivers/ide | |
parent | b5479167f4206e0d821a51ae149d921cd7a58e54 (diff) |
ide: decrease size of ->pc_buf field in struct ide_atapi_pc
struct ide_atapi_pc is often allocated on the stack and size of ->pc_buf
size is 256 bytes. However since only ide_floppy_create_read_capacity_cmd()
and idetape_create_inquiry_cmd() require such size allocate buffers for
these pc-s explicitely and decrease ->pc_buf size to 64 bytes.
Cc: Borislav Petkov <petkovbb@gmail.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide')
-rw-r--r-- | drivers/ide/ide-floppy.c | 5 | ||||
-rw-r--r-- | drivers/ide/ide-floppy_ioctl.c | 5 | ||||
-rw-r--r-- | drivers/ide/ide-tape.c | 4 |
3 files changed, 12 insertions, 2 deletions
diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 7ae66233483..0faae309829 100644 --- a/drivers/ide/ide-floppy.c +++ b/drivers/ide/ide-floppy.c | |||
@@ -385,7 +385,7 @@ static int ide_floppy_get_capacity(ide_drive_t *drive) | |||
385 | struct gendisk *disk = floppy->disk; | 385 | struct gendisk *disk = floppy->disk; |
386 | struct ide_atapi_pc pc; | 386 | struct ide_atapi_pc pc; |
387 | u8 *cap_desc; | 387 | u8 *cap_desc; |
388 | u8 header_len, desc_cnt; | 388 | u8 pc_buf[256], header_len, desc_cnt; |
389 | int i, rc = 1, blocks, length; | 389 | int i, rc = 1, blocks, length; |
390 | 390 | ||
391 | drive->bios_cyl = 0; | 391 | drive->bios_cyl = 0; |
@@ -395,6 +395,9 @@ static int ide_floppy_get_capacity(ide_drive_t *drive) | |||
395 | drive->capacity64 = 0; | 395 | drive->capacity64 = 0; |
396 | 396 | ||
397 | ide_floppy_create_read_capacity_cmd(&pc); | 397 | ide_floppy_create_read_capacity_cmd(&pc); |
398 | pc.buf = &pc_buf[0]; | ||
399 | pc.buf_size = sizeof(pc_buf); | ||
400 | |||
398 | if (ide_queue_pc_tail(drive, disk, &pc)) { | 401 | if (ide_queue_pc_tail(drive, disk, &pc)) { |
399 | printk(KERN_ERR PFX "Can't get floppy parameters\n"); | 402 | printk(KERN_ERR PFX "Can't get floppy parameters\n"); |
400 | return 1; | 403 | return 1; |
diff --git a/drivers/ide/ide-floppy_ioctl.c b/drivers/ide/ide-floppy_ioctl.c index 8f8be854603..cd8a42027ed 100644 --- a/drivers/ide/ide-floppy_ioctl.c +++ b/drivers/ide/ide-floppy_ioctl.c | |||
@@ -36,9 +36,9 @@ static int ide_floppy_get_format_capacities(ide_drive_t *drive, | |||
36 | int __user *arg) | 36 | int __user *arg) |
37 | { | 37 | { |
38 | struct ide_disk_obj *floppy = drive->driver_data; | 38 | struct ide_disk_obj *floppy = drive->driver_data; |
39 | u8 header_len, desc_cnt; | ||
40 | int i, blocks, length, u_array_size, u_index; | 39 | int i, blocks, length, u_array_size, u_index; |
41 | int __user *argp; | 40 | int __user *argp; |
41 | u8 pc_buf[256], header_len, desc_cnt; | ||
42 | 42 | ||
43 | if (get_user(u_array_size, arg)) | 43 | if (get_user(u_array_size, arg)) |
44 | return -EFAULT; | 44 | return -EFAULT; |
@@ -47,6 +47,9 @@ static int ide_floppy_get_format_capacities(ide_drive_t *drive, | |||
47 | return -EINVAL; | 47 | return -EINVAL; |
48 | 48 | ||
49 | ide_floppy_create_read_capacity_cmd(pc); | 49 | ide_floppy_create_read_capacity_cmd(pc); |
50 | pc->buf = &pc_buf[0]; | ||
51 | pc->buf_size = sizeof(pc_buf); | ||
52 | |||
50 | if (ide_queue_pc_tail(drive, floppy->disk, pc)) { | 53 | if (ide_queue_pc_tail(drive, floppy->disk, pc)) { |
51 | printk(KERN_ERR "ide-floppy: Can't get floppy parameters\n"); | 54 | printk(KERN_ERR "ide-floppy: Can't get floppy parameters\n"); |
52 | return -EIO; | 55 | return -EIO; |
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c index 64dfa7458f8..cafc67d9e2e 100644 --- a/drivers/ide/ide-tape.c +++ b/drivers/ide/ide-tape.c | |||
@@ -2014,9 +2014,13 @@ static void idetape_get_inquiry_results(ide_drive_t *drive) | |||
2014 | { | 2014 | { |
2015 | idetape_tape_t *tape = drive->driver_data; | 2015 | idetape_tape_t *tape = drive->driver_data; |
2016 | struct ide_atapi_pc pc; | 2016 | struct ide_atapi_pc pc; |
2017 | u8 pc_buf[256]; | ||
2017 | char fw_rev[4], vendor_id[8], product_id[16]; | 2018 | char fw_rev[4], vendor_id[8], product_id[16]; |
2018 | 2019 | ||
2019 | idetape_create_inquiry_cmd(&pc); | 2020 | idetape_create_inquiry_cmd(&pc); |
2021 | pc.buf = &pc_buf[0]; | ||
2022 | pc.buf_size = sizeof(pc_buf); | ||
2023 | |||
2020 | if (ide_queue_pc_tail(drive, tape->disk, &pc)) { | 2024 | if (ide_queue_pc_tail(drive, tape->disk, &pc)) { |
2021 | printk(KERN_ERR "ide-tape: %s: can't get INQUIRY results\n", | 2025 | printk(KERN_ERR "ide-tape: %s: can't get INQUIRY results\n", |
2022 | tape->name); | 2026 | tape->name); |