Home » Mailing list archives » linux-kernel » 2007 » October » 17

[PATCH 2/3] ide: Fix siimage driver accessing beyond array boundary

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Benjamin Herrenschmidt <benh@...>
To: Bartlomiej Zolnierkiewicz <bzolnier@...>
Cc: Linux IDE <linux-ide@...>, <linux-kernel@...>, Andrew Morton <akpm@...>
Subject: [PATCH 2/3] ide: Fix siimage driver accessing beyond array boundary
Date: Wednesday, October 17, 2007 - 8:56 pm

The siimage use an incorrect construct to access the other drive
of a pair, causing it to access beyond an array boundary on non-0
interfaces. This fixes it by using the new ide_get_paired_drive()
hepler instead.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
---

 drivers/ide/pci/siimage.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux-work/drivers/ide/pci/siimage.c
===================================================================
--- linux-work.orig/drivers/ide/pci/siimage.c	2007-10-18 10:42:56.000000000 +1000
+++ linux-work/drivers/ide/pci/siimage.c	2007-10-18 10:43:09.000000000 +1000
@@ -180,7 +180,7 @@ static void sil_set_pio_mode(ide_drive_t
 	const u16 data_speed[]	= { 0x328a, 0x2283, 0x1104, 0x10c3, 0x10c1 };
 
 	ide_hwif_t *hwif	= HWIF(drive);
-	ide_drive_t *pair	= &hwif->drives[drive->dn ^ 1];
+	ide_drive_t *pair	= ide_get_paired_drive(drive);
 	u32 speedt		= 0;
 	u16 speedp		= 0;
 	unsigned long addr	= siimage_seldev(drive, 0x04);
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 0/3] ide: Fix use of paired device, Benjamin Herrenschmidt, (Wed Oct 17, 8:56 pm)
Re: [PATCH 0/3] ide: Fix use of paired device, Bartlomiej Zolnierkiewicz..., (Thu Oct 18, 4:29 pm)
Re: [PATCH 0/3] ide: Fix use of paired device, Sergei Shtylyov, (Thu Oct 18, 7:54 am)
Re: [PATCH 0/3] ide: Fix use of paired device, Benjamin Herrenschmidt, (Thu Oct 18, 8:14 am)
Re: [PATCH 0/3] ide: Fix use of paired device, Sergei Shtylyov, (Thu Oct 18, 7:58 am)
[PATCH 3/3] ide: Fix cs5535 driver accessing beyond array bo..., Benjamin Herrenschmidt, (Wed Oct 17, 8:56 pm)
Re: [PATCH 3/3] ide: Fix cs5535 driver accessing beyond arra..., Bartlomiej Zolnierkiewicz..., (Thu Oct 18, 5:10 pm)
Re: [PATCH 3/3] ide: Fix cs5535 driver accessing beyond arra..., Sergei Shtylyov, (Thu Oct 18, 8:03 am)
[PATCH 2/3] ide: Fix siimage driver accessing beyond array b..., Benjamin Herrenschmidt, (Wed Oct 17, 8:56 pm)
Re: [PATCH 2/3] ide: Fix siimage driver accessing beyond arr..., Bartlomiej Zolnierkiewicz..., (Thu Oct 18, 5:10 pm)
Re: [PATCH 2/3] ide: Fix siimage driver accessing beyond arr..., Sergei Shtylyov, (Thu Oct 18, 8:02 am)
Re: [PATCH 2/3] ide: Fix siimage driver accessing beyond arr..., Benjamin Herrenschmidt, (Thu Oct 18, 8:16 am)
Re: [PATCH 2/3] ide: Fix siimage driver accessing beyond arr..., Bartlomiej Zolnierkiewicz..., (Thu Oct 18, 5:30 pm)
[PATCH 1/3] ide: Add ide_get_paired_drive() helper, Benjamin Herrenschmidt, (Wed Oct 17, 8:56 pm)
Re: [PATCH 1/3] ide: Add ide_get_paired_drive() helper, Bartlomiej Zolnierkiewicz..., (Thu Oct 18, 5:10 pm)
Re: [PATCH 1/3] ide: Add ide_get_paired_drive() helper, Sergei Shtylyov, (Thu Oct 18, 8:00 am)