From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Hogan Subject: Re: [BUG] NULL pointer dereference in udf_sb_free_partitions Date: Mon, 14 Jan 2013 21:06:37 +0000 Message-ID: <20130114210637.GA2122@balrog> References: <20130112220015.GA2387@balrog> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kara , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Namjae Jeon Return-path: Received: from mail-wg0-f46.google.com ([74.125.82.46]:43331 "EHLO mail-wg0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757687Ab3ANVGm (ORCPT ); Mon, 14 Jan 2013 16:06:42 -0500 Received: by mail-wg0-f46.google.com with SMTP id dr13so2238320wgb.1 for ; Mon, 14 Jan 2013 13:06:41 -0800 (PST) Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Hi, On Mon, Jan 14, 2013 at 02:19:39PM +0900, Namjae Jeon wrote: > There is missing exception handling in memory leak patch. (udf: Fix > memory leak when mounting) > So, Would you try to reproduce this problem with the below patch ? > > Thanks. > > --------------------------------------------------------------------------- > Subject: [PATCH] UDF: Fix a null pointer dereference in udf_sb_free_partitions > > This patch fixes a regression caused by commit bff943af6fe > "udf: Fix memory leak when mounting" due to which it was triggering > a kernel null point dereference in case of mount failed OR when allocating > memory to sbi->s_partmaps failed in function udf_sb_alloc_partition_maps. > > Reported-by: James Hogan > Signed-off-by: Namjae Jeon > Signed-off-by: Ashish Sangwan v3.8-rc3 + your patch no longer crashes. Tested-by: James Hogan Thanks James