diff options
author | Eric Van Hensbergen <ericvh@gmail.com> | 2007-01-26 03:57:04 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-01-26 16:50:59 -0500 |
commit | 621997cd392a541bbe964e7521e2e4b2a7a03c4e (patch) | |
tree | b9ece289d5ff4d69616c02510edf2c00a858ccfd | |
parent | f94b347059809aecd97af53a084fe5f3db045d92 (diff) |
[PATCH] 9p: fix rename return code
9p doesn't handle renames between directories -- however, we were returning
EPERM instead of EXDEV when we detected this case.
Signed-off-by: Eric Van Hensbergren <ericvh@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/9p/vfs_inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c index 18f26cdfd882..05d30e89ba45 100644 --- a/fs/9p/vfs_inode.c +++ b/fs/9p/vfs_inode.c | |||
@@ -767,7 +767,7 @@ v9fs_vfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
767 | /* 9P can only handle file rename in the same directory */ | 767 | /* 9P can only handle file rename in the same directory */ |
768 | if (memcmp(&olddirfid->qid, &newdirfid->qid, sizeof(newdirfid->qid))) { | 768 | if (memcmp(&olddirfid->qid, &newdirfid->qid, sizeof(newdirfid->qid))) { |
769 | dprintk(DEBUG_ERROR, "old dir and new dir are different\n"); | 769 | dprintk(DEBUG_ERROR, "old dir and new dir are different\n"); |
770 | retval = -EPERM; | 770 | retval = -EXDEV; |
771 | goto FreeFcallnBail; | 771 | goto FreeFcallnBail; |
772 | } | 772 | } |
773 | 773 | ||