Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5fc242c1 authored by Al Viro's avatar Al Viro Committed by Greg Kroah-Hartman
Browse files

nfsd: lock_rename() needs both directories to live on the same fs 



commit 1aee9158bc978f91701c5992e395efbc6da2de3c upstream.

... checking that after lock_rename() is too late.  Incidentally,
NFSv2 had no nfserr_xdev...

Fixes: aa387d6c "nfsd: fix EXDEV checking in rename"
Cc: stable@vger.kernel.org # v3.9+
Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
Acked-by: default avatarChuck Lever <chuck.lever@oracle.com>
Tested-by: default avatarJeff Layton <jlayton@kernel.org>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e9a988cd

fs/nfsd/vfs.c

+6 −6
Original line number Diff line number Diff line
@@ -1689,6 +1689,12 @@ nfsd_rename(struct svc_rqst *rqstp, struct svc_fh *ffhp, char *fname, int flen, 
	if (!flen || isdotent(fname, flen) || !tlen || isdotent(tname, tlen))

		goto out;



	err = (rqstp->rq_vers == 2) ? nfserr_acces : nfserr_xdev;

	if (ffhp->fh_export->ex_path.mnt != tfhp->fh_export->ex_path.mnt)

		goto out;

	if (ffhp->fh_export->ex_path.dentry != tfhp->fh_export->ex_path.dentry)

		goto out;



retry:

	host_err = fh_want_write(ffhp);

	if (host_err) {
@@ -1723,12 +1729,6 @@ nfsd_rename(struct svc_rqst *rqstp, struct svc_fh *ffhp, char *fname, int flen, 
	if (ndentry == trap)

		goto out_dput_new;



	host_err = -EXDEV;

	if (ffhp->fh_export->ex_path.mnt != tfhp->fh_export->ex_path.mnt)

		goto out_dput_new;

	if (ffhp->fh_export->ex_path.dentry != tfhp->fh_export->ex_path.dentry)

		goto out_dput_new;



	if (nfsd_has_cached_files(ndentry)) {

		has_cached = true;

		goto out_dput_old;