From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 3/3] ide: Fix cs5535 driver accessing beyond array boundary Date: Thu, 18 Oct 2007 16:03:49 +0400 Message-ID: <47174BA5.7040700@ru.mvista.com> References: <20071018005624.DC749DDEAB@ozlabs.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from h155.mvista.com ([63.81.120.155]:42714 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1761811AbXJRMDh (ORCPT ); Thu, 18 Oct 2007 08:03:37 -0400 In-Reply-To: <20071018005624.DC749DDEAB@ozlabs.org> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Benjamin Herrenschmidt Cc: Bartlomiej Zolnierkiewicz , Linux IDE , linux-kernel@vger.kernel.org, Andrew Morton Benjamin Herrenschmidt wrote: > The cs5535 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 Please update the driver version in the driver's heading comment. It also would be preferrable to fold this patch together with siimage path, or even fold all three together. Otherwise Acked-by: Sergei Shtylyov MBR, Sergei