From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.174]) by ozlabs.org (Postfix) with ESMTP id 06D82DDE02 for ; Thu, 3 Apr 2008 10:39:18 +1100 (EST) From: Arnd Bergmann To: linuxppc-dev@ozlabs.org Subject: Re: [PATCH 1/3][POWERPC][V2] of_serial: Fix possible null dereference. Date: Thu, 3 Apr 2008 01:39:04 +0200 References: <20080402165216.8FD2BC9807F@mail21-dub.bigfish.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200804030139.05497.arnd@arndb.de> Cc: Paul Mackerras , John Linn List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wednesday 02 April 2008, Grant Likely wrote: > On Wed, Apr 2, 2008 at 10:52 AM, John Linn wrote: > > From: Stephen Neuendorffer > > > > =A0The of_serial driver queries the current-speed property and attempts > > =A0to use it to register the custom_divisor property of the uart_port. > > =A0However, if current-speed is not set, then this code will dereference > > =A0a bad pointer. =A0The fix is to only set custom_divisor when a > > =A0current-speed property appears in the device tree. > > > > =A0Signed-off-by: Stephen Neuendorffer > > =A0Signed-off-by: John Linn > Acked-by: Grant Likely Acked-by: Arnd Bergmann