OSDN Git Service

nfsd: special case truncates some more
authorChristoph Hellwig <hch@lst.de>
Mon, 20 Feb 2017 06:21:33 +0000 (07:21 +0100)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 21 Feb 2017 15:13:37 +0000 (10:13 -0500)
commit783112f7401ff449d979530209b3f6c2594fdb4e
tree82ecfc2b095ecd229746b57100bc789bd403a009
parent758e99fefe1d9230111296956335cd35995c0eaf
nfsd: special case truncates some more

Both the NFS protocols and the Linux VFS use a setattr operation with a
bitmap of attributes to set to set various file attributes including the
file size and the uid/gid.

The Linux syscalls never mix size updates with unrelated updates like
the uid/gid, and some file systems like XFS and GFS2 rely on the fact
that truncates don't update random other attributes, and many other file
systems handle the case but do not update the other attributes in the
same transaction.  NFSD on the other hand passes the attributes it gets
on the wire more or less directly through to the VFS, leading to updates
the file systems don't expect.  XFS at least has an assert on the
allowed attributes, which caught an unusual NFS client setting the size
and group at the same time.

To handle this issue properly this splits the notify_change call in
nfsd_setattr into two separate ones.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Cc: stable@vger.kernel.org
Tested-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/vfs.c