From: Trond Myklebust Subject: Re: [GIT] More NFS client fixes for 2.6.24-rc6 Date: Thu, 03 Jan 2008 09:45:57 -0500 Message-ID: <1199371557.7357.14.camel@heimdal.trondhjem.org> References: <1199329417.28241.3.camel@heimdal.trondhjem.org> <20080103093048.GA16803@elte.hu> Mime-Version: 1.0 Content-Type: text/plain Cc: Linus Torvalds , Andrew Morton , linux-kernel@vger.kernel.org, linux-nfs@vger.kernel.org To: Ingo Molnar Return-path: Received: from mx2.netapp.com ([216.240.18.37]:29418 "EHLO mx2.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751212AbYACOqE (ORCPT ); Thu, 3 Jan 2008 09:46:04 -0500 In-Reply-To: <20080103093048.GA16803@elte.hu> Sender: linux-nfs-owner@vger.kernel.org List-ID: On Thu, 2008-01-03 at 10:30 +0100, Ingo Molnar wrote: > * Trond Myklebust wrote: > > > commit 53478daff2c8b494d2af1ede6611f166f81bc393 > > Author: Trond Myklebust > > Date: Wed Jan 2 13:28:57 2008 -0500 > > > > NFS: Fix a possible Oops in fs/nfs/super.c > > > > Sigh... commit 4584f520e1f773082ef44ff4f8969a5d992b16ec (NFS: Fix NFS > > mountpoint crossing...) had a slight flaw: server can be NULL if sget() > > returned an existing superblock. > > > > Fix the fix by dereferencing s->s_fs_info. > > > > Also add in the same namespace Oops fix for NFSv4 in both the mountpoint > > crossing case, and the referral case. > > > > Signed-off-by: Trond Myklebust > > shouldnt this commit have included the full credit of the bugfix: > > http://bugzilla.kernel.org/show_bug.cgi?id=9647 > > ------------> > Description From Adrian Bunk 2007-12-27 12:36 > > The Coverity checker spotted that commit > 4584f520e1f773082ef44ff4f8969a5d992b16ec introduced the following NULL > dereference in 2.6.24-rc6: Point taken, however I assume that a reference to the bugzilla report should suffice. I've therefore updated the commit changelog to read as follows: NFS: Fix a possible Oops in fs/nfs/super.c Sigh... commit 4584f520e1f773082ef44ff4f8969a5d992b16ec (NFS: Fix NFS mountpoint crossing...) had a slight flaw: server can be NULL if sget() returned an existing superblock. Fix the fix by dereferencing s->s_fs_info. Thanks to Coverity/Adrian Bunk and Frank Filz for spotting the bug. (See http://bugzilla.kernel.org/show_bug.cgi?id=9647) Also add in the same namespace Oops fix for NFSv4 in both the mountpoint crossing case, and the referral case. Signed-off-by: Trond Myklebust Cheers Trond