[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#659111: Regarding NFSv4: Save the owner/group name string when doing open



On Tue, Apr 24, 2012 at 10:04:50PM +0200, Rik Theys wrote:
> Hi,
> 
> I'm experiencing the bug described in the Debian[1] and Red Hat[2] bug tracker.
> 
> This bug seems to have been fixed in the 3.3 kernel with your commit
> 6926afd1925a54a13684ebe05987868890665e2b:
> 
> From: Trond Myklebust <Trond.Myklebust@netapp.com>
> Date: Sat, 7 Jan 2012 13:22:46 -0500
> Subject: NFSv4: Save the owner/group name string when doing open
> 
> commit 6926afd1925a54a13684ebe05987868890665e2b upstream.
> 
> ...so that we can do the uid/gid mapping outside the asynchronous RPC
> context.
> This fixes a bug in the current NFSv4 atomic open code where the client
> isn't able to determine what the true uid/gid fields of the file are,
> (because the asynchronous nature of the OPEN call denies it the ability
> to do an upcall) and so fills them with default values, marking the
> inode as needing revalidation.
> Unfortunately, in some cases, the VFS will do some additional sanity
> checks on the file, and may override the server's decision to allow
> the open because it sees the wrong owner/group fields.
> 
> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
> 
> It seems Red Hat will apply the patch in their RHEL 6.3 kernel. I would
> also like to see this patch included in the upcoming Debian 7.0 kernel,
> which is based on kernel 3.2.
> 
> I would like to propose this patch for a stable kernel update (3.2.x
> and/or 3.0.x). Trond, do you agree that this patch (alone)
> can/should be part of a stable update? The Debian maintainers would
> prefer to see the patch be part of a stable update to consider it.

This patch seems to violate Documentatin/stable_kernel_rules.txt,
doesn't it?  And as such, how could I include it in any stable kernel
release?

thanks,

greg k-h



Reply to: