Commit f0edef8c authored by Grant Likely's avatar Grant Likely Committed by Bartlomiej Zolnierkiewicz

xsysace: Fix dereferencing of cf_id after hd_driveid removal

Commit 4aaf2fec (xsysace: make it
'struct hd_driveid'-free) converted the cf_id member of 'struct
ace_device' from a 'struct hd_driveid' to a u16 array.  However,
references to the base of the structure were still using the '&'
operator.  When the address was used with the ata_id_u32() macro, the
compiler used the size of the entire array instead of sizeof(u16) to
calculate the offset from the base address.

This patch removes the use of the '&' operator from all references of
cf_id to fix the bug and remove future confusion.
Signed-off-by: default avatarGrant Likely <grant.likely@secretlab.ca>
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
parent fb4252e5
...@@ -563,7 +563,7 @@ static void ace_fsm_dostate(struct ace_device *ace) ...@@ -563,7 +563,7 @@ static void ace_fsm_dostate(struct ace_device *ace)
case ACE_FSM_STATE_IDENTIFY_PREPARE: case ACE_FSM_STATE_IDENTIFY_PREPARE:
/* Send identify command */ /* Send identify command */
ace->fsm_task = ACE_TASK_IDENTIFY; ace->fsm_task = ACE_TASK_IDENTIFY;
ace->data_ptr = &ace->cf_id; ace->data_ptr = ace->cf_id;
ace->data_count = ACE_BUF_PER_SECTOR; ace->data_count = ACE_BUF_PER_SECTOR;
ace_out(ace, ACE_SECCNTCMD, ACE_SECCNTCMD_IDENTIFY); ace_out(ace, ACE_SECCNTCMD, ACE_SECCNTCMD_IDENTIFY);
...@@ -608,8 +608,8 @@ static void ace_fsm_dostate(struct ace_device *ace) ...@@ -608,8 +608,8 @@ static void ace_fsm_dostate(struct ace_device *ace)
break; break;
case ACE_FSM_STATE_IDENTIFY_COMPLETE: case ACE_FSM_STATE_IDENTIFY_COMPLETE:
ace_fix_driveid(&ace->cf_id[0]); ace_fix_driveid(ace->cf_id);
ace_dump_mem(&ace->cf_id, 512); /* Debug: Dump out disk ID */ ace_dump_mem(ace->cf_id, 512); /* Debug: Dump out disk ID */
if (ace->data_result) { if (ace->data_result) {
/* Error occured, disable the disk */ /* Error occured, disable the disk */
...@@ -622,9 +622,9 @@ static void ace_fsm_dostate(struct ace_device *ace) ...@@ -622,9 +622,9 @@ static void ace_fsm_dostate(struct ace_device *ace)
/* Record disk parameters */ /* Record disk parameters */
set_capacity(ace->gd, set_capacity(ace->gd,
ata_id_u32(&ace->cf_id, ATA_ID_LBA_CAPACITY)); ata_id_u32(ace->cf_id, ATA_ID_LBA_CAPACITY));
dev_info(ace->dev, "capacity: %i sectors\n", dev_info(ace->dev, "capacity: %i sectors\n",
ata_id_u32(&ace->cf_id, ATA_ID_LBA_CAPACITY)); ata_id_u32(ace->cf_id, ATA_ID_LBA_CAPACITY));
} }
/* We're done, drop to IDLE state and notify waiters */ /* We're done, drop to IDLE state and notify waiters */
...@@ -923,7 +923,7 @@ static int ace_release(struct gendisk *disk, fmode_t mode) ...@@ -923,7 +923,7 @@ static int ace_release(struct gendisk *disk, fmode_t mode)
static int ace_getgeo(struct block_device *bdev, struct hd_geometry *geo) static int ace_getgeo(struct block_device *bdev, struct hd_geometry *geo)
{ {
struct ace_device *ace = bdev->bd_disk->private_data; struct ace_device *ace = bdev->bd_disk->private_data;
u16 *cf_id = &ace->cf_id[0]; u16 *cf_id = ace->cf_id;
dev_dbg(ace->dev, "ace_getgeo()\n"); dev_dbg(ace->dev, "ace_getgeo()\n");
......
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