diff options
| author | Casey Bodley <cbodley@citi.umich.edu> | 2011-06-03 12:21:23 -0400 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2011-06-06 18:46:56 -0400 |
| commit | 7d751f6f8c679f51b73d01a1b5269347a929004c (patch) | |
| tree | d3197afcf1f8f241000eca04c45735fff973e70d /fs/nfsd/vfs.c | |
| parent | be1f4084b4824301e640e81d63b6275cd99ee6a1 (diff) | |
| download | lwn-7d751f6f8c679f51b73d01a1b5269347a929004c.tar.gz lwn-7d751f6f8c679f51b73d01a1b5269347a929004c.zip | |
nfsd: link returns nfserr_delay when breaking lease
fix for commit 4795bb37effb7b8fe77e2d2034545d062d3788a8, nfsd: break
lease on unlink, link, and rename
if the LINK operation breaks a delegation, it returns NFS4ERR_NOENT
(which is not a valid error in rfc 5661) instead of NFS4ERR_DELAY.
the return value of nfsd_break_lease() in nfsd_link() must be
converted from host_err to err
Signed-off-by: Casey Bodley <cbodley@citi.umich.edu>
Cc: stable@kernel.org
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/vfs.c')
| -rw-r--r-- | fs/nfsd/vfs.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index d5718273bb32..f3fb61b69a1e 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c @@ -1660,8 +1660,10 @@ nfsd_link(struct svc_rqst *rqstp, struct svc_fh *ffhp, if (!dold->d_inode) goto out_drop_write; host_err = nfsd_break_lease(dold->d_inode); - if (host_err) + if (host_err) { + err = nfserrno(host_err); goto out_drop_write; + } host_err = vfs_link(dold, dirp, dnew); if (!host_err) { err = nfserrno(commit_metadata(ffhp)); |