diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2005-08-25 16:25:34 -0700 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2005-09-23 12:37:58 -0400 |
| commit | 20509f1bc553ed7fafa88fa8d01c6212d1876d9f (patch) | |
| tree | da10ad62009995269dc0733a254a0bc1f6293aa2 /fs/nfs | |
| parent | 12f44f46bc9c6dc79255e661b085797af395d8da (diff) | |
| download | lwn-20509f1bc553ed7fafa88fa8d01c6212d1876d9f.tar.gz lwn-20509f1bc553ed7fafa88fa8d01c6212d1876d9f.zip | |
NFS: Drop inode after rename
When doing a rename on top of an existing file that is not in use,
the inode of the overwritten file will remain in the icache.
The fix is to decrement i_nlink of the overwritten inode, like we
do for unlink, rmdir etc already.
Problem diagnosed by Olaf Kirch. This patch is a slight variation
on his fix.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs')
| -rw-r--r-- | fs/nfs/dir.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c index 2df639f143e8..94a7fcee0624 100644 --- a/fs/nfs/dir.c +++ b/fs/nfs/dir.c @@ -1539,7 +1539,8 @@ static int nfs_rename(struct inode *old_dir, struct dentry *old_dentry, #endif goto out; } - } + } else + new_inode->i_nlink--; go_ahead: /* |