summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorOGAWA Hirofumi <hirofumi@mail.parknet.co.jp>2010-01-06 18:48:26 -0500
committerTrond Myklebust <Trond.Myklebust@netapp.com>2010-01-06 18:48:26 -0500
commit56335936de1a41c8978fde62b2158af77ddc7258 (patch)
treef0061bef629a1c6a14e08f1b660b3beca0adc7cd /fs
parent6c8530993e1fdf1d6af0403e796fe14d80b4b097 (diff)
downloadop-kernel-dev-56335936de1a41c8978fde62b2158af77ddc7258.zip
op-kernel-dev-56335936de1a41c8978fde62b2158af77ddc7258.tar.gz
nfs: fix oops in nfs_rename()
Recent change is missing to update "rehash". With that change, it will become the cause of adding dentry to hash twice. This explains the reason of Oops (dereference the freed dentry in __d_lookup()) on my machine. Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> Reported-by: Marvin <marvin24@gmx.de> Cc: Trond Myklebust <trond.myklebust@fys.uio.no> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs')
-rw-r--r--fs/nfs/dir.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c
index 2c5ace4..3c7f03b 100644
--- a/fs/nfs/dir.c
+++ b/fs/nfs/dir.c
@@ -1615,6 +1615,7 @@ static int nfs_rename(struct inode *old_dir, struct dentry *old_dentry,
goto out;
new_dentry = dentry;
+ rehash = NULL;
new_inode = NULL;
}
}
OpenPOWER on IntegriCloud