Commit 801bd32e authored by Borislav Petkov's avatar Borislav Petkov Committed by Bartlomiej Zolnierkiewicz

ide-tape: fix vendor strings

Remove superfluous two bytes from each string buffer and add proper length
format specifiers.
Signed-off-by: default avatarBorislav Petkov <petkovbb@gmail.com>
Tested-by: default avatarMark de Wever <koraq@xs4all.nl>
Acked-by: default avatarSergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
parent 3d977760
...@@ -2338,7 +2338,7 @@ static void idetape_get_inquiry_results(ide_drive_t *drive) ...@@ -2338,7 +2338,7 @@ static void idetape_get_inquiry_results(ide_drive_t *drive)
{ {
idetape_tape_t *tape = drive->driver_data; idetape_tape_t *tape = drive->driver_data;
struct ide_atapi_pc pc; struct ide_atapi_pc pc;
char fw_rev[6], vendor_id[10], product_id[18]; char fw_rev[4], vendor_id[8], product_id[16];
idetape_create_inquiry_cmd(&pc); idetape_create_inquiry_cmd(&pc);
if (idetape_queue_pc_tail(drive, &pc)) { if (idetape_queue_pc_tail(drive, &pc)) {
...@@ -2350,11 +2350,11 @@ static void idetape_get_inquiry_results(ide_drive_t *drive) ...@@ -2350,11 +2350,11 @@ static void idetape_get_inquiry_results(ide_drive_t *drive)
memcpy(product_id, &pc.buf[16], 16); memcpy(product_id, &pc.buf[16], 16);
memcpy(fw_rev, &pc.buf[32], 4); memcpy(fw_rev, &pc.buf[32], 4);
ide_fixstring(vendor_id, 10, 0); ide_fixstring(vendor_id, 8, 0);
ide_fixstring(product_id, 18, 0); ide_fixstring(product_id, 16, 0);
ide_fixstring(fw_rev, 6, 0); ide_fixstring(fw_rev, 4, 0);
printk(KERN_INFO "ide-tape: %s <-> %s: %s %s rev %s\n", printk(KERN_INFO "ide-tape: %s <-> %s: %.8s %.16s rev %.4s\n",
drive->name, tape->name, vendor_id, product_id, fw_rev); drive->name, tape->name, vendor_id, product_id, fw_rev);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment