nfsd: avoid permission checks on EXCLUSIVE_CREATE replay
authorNeil Brown <neilb@suse.de>
Fri, 7 Dec 2012 20:40:55 +0000 (15:40 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Mon, 10 Dec 2012 21:25:31 +0000 (16:25 -0500)
commit7007c90fb9fef593b4aeaeee57e6a6754276c97c
tree8fe1b5daa2132876b364801aaef4127774c6c4b3
parent756933ee8a75970ca7d8a10e922f5d44720457a4
nfsd: avoid permission checks on EXCLUSIVE_CREATE replay

With NFSv4, if we create a file then open it we explicit avoid checking
the permissions on the file during the open because the fact that we
created it ensures we should be allow to open it (the create and the
open should appear to be a single operation).

However if the reply to an EXCLUSIVE create gets lots and the client
resends the create, the current code will perform the permission check -
because it doesn't realise that it did the open already..

This patch should fix this.

Note that I haven't actually seen this cause a problem.  I was just
looking at the code trying to figure out a different EXCLUSIVE open
related issue, and this looked wrong.

(Fix confirmed with pynfs 4.0 test OPEN4--bfields)

Cc: stable@kernel.org
Signed-off-by: NeilBrown <neilb@suse.de>
[bfields: use OWNER_OVERRIDE and update for 4.1]
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4proc.c
fs/nfsd/vfs.c