[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



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.

Regards,

Rik


[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=659111
[2] https://bugzilla.redhat.com/show_bug.cgi?format=multiple&id=789298

--
Rik Theys
Senior System Engineer
KU Leuven - Dept. Elektrotechniek (ESAT)
Kasteelpark Arenberg 10
B-3001 LEUVEN - HEVERLEE
Tel.: +32(0)16/32.11.07
----------------------------------------------------------------
<<Any errors in spelling, tact or fact are transmission errors>>



Reply to: