diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-01-07 13:22:46 -0500 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-01-07 13:22:46 -0500 |
| commit | 6926afd1925a54a13684ebe05987868890665e2b (patch) | |
| tree | 14dd0c00c757277d6f28b732c08206376e6dbf31 /fs/nfs/inode.c | |
| parent | e2fecb215b321db0e4a5b2597349a63c07bec42f (diff) | |
| download | op-kernel-dev-6926afd1925a54a13684ebe05987868890665e2b.zip op-kernel-dev-6926afd1925a54a13684ebe05987868890665e2b.tar.gz | |
NFSv4: Save the owner/group name string when doing open
...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>
Diffstat (limited to 'fs/nfs/inode.c')
| -rw-r--r-- | fs/nfs/inode.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/nfs/inode.c b/fs/nfs/inode.c index 50a15fa..f59cab1 100644 --- a/fs/nfs/inode.c +++ b/fs/nfs/inode.c @@ -1019,6 +1019,8 @@ void nfs_fattr_init(struct nfs_fattr *fattr) fattr->valid = 0; fattr->time_start = jiffies; fattr->gencount = nfs_inc_attr_generation_counter(); + fattr->owner_name = NULL; + fattr->group_name = NULL; } struct nfs_fattr *nfs_alloc_fattr(void) |
