diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2011-07-21 15:49:09 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-07-22 19:42:11 -0400 |
commit | 841590ce16c19a3ce38028adfc8b1955482ee00c (patch) | |
tree | 83f20931de0103f1b3b60f6ba4b8aa8490c08084 | |
parent | ed70afcd6e795e3de98df56f1cd0f898fbf641a7 (diff) |
fix IN_DELETE_SELF on overwriting rename() on ramfs et.al.
On ramfs and other simple_rename() users IN_DELETE_SELF is not generated
for victim of overwriting rename() if it's is a directory. Works on
most of the local filesystems and really trivial to fix...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/libfs.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/libfs.c b/fs/libfs.c index 8f2271a5df53..c18e9a1235b6 100644 --- a/fs/libfs.c +++ b/fs/libfs.c | |||
@@ -328,8 +328,10 @@ int simple_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
328 | 328 | ||
329 | if (new_dentry->d_inode) { | 329 | if (new_dentry->d_inode) { |
330 | simple_unlink(new_dir, new_dentry); | 330 | simple_unlink(new_dir, new_dentry); |
331 | if (they_are_dirs) | 331 | if (they_are_dirs) { |
332 | drop_nlink(new_dentry->d_inode); | ||
332 | drop_nlink(old_dir); | 333 | drop_nlink(old_dir); |
334 | } | ||
333 | } else if (they_are_dirs) { | 335 | } else if (they_are_dirs) { |
334 | drop_nlink(old_dir); | 336 | drop_nlink(old_dir); |
335 | inc_nlink(new_dir); | 337 | inc_nlink(new_dir); |