diff options
author | Erez Zadok <ezk@cs.sunysb.edu> | 2009-12-06 18:05:30 -0500 |
---|---|---|
committer | Tyler Hicks <tyhicks@linux.vnet.ibm.com> | 2010-01-19 23:36:00 -0500 |
commit | c44a66d674688f1e1d0b2f6f56bd9c6a1b061cae (patch) | |
tree | 52ff029a1c19b11b75ccc25ed7a5e00619e05800 /fs | |
parent | 0d132f7364694da8f7cafd49e2fc2721b73e96e4 (diff) |
ecryptfs: remove unnecessary d_drop calls in ecryptfs_link
Unnecessary because it would unhash perfectly valid dentries, causing them
to have to be re-looked up the next time they're needed, which presumably is
right after.
Signed-off-by: Aseem Rastogi <arastogi@cs.sunysb.edu>
Signed-off-by: Shrikar archak <shrikar84@gmail.com>
Signed-off-by: Erez Zadok <ezk@cs.sunysb.edu>
Cc: Saumitra Bhanage <sbhanage@cs.sunysb.edu>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Tyler Hicks <tyhicks@linux.vnet.ibm.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ecryptfs/inode.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/ecryptfs/inode.c b/fs/ecryptfs/inode.c index 764dc777b268..e03415293625 100644 --- a/fs/ecryptfs/inode.c +++ b/fs/ecryptfs/inode.c | |||
@@ -463,9 +463,6 @@ out_lock: | |||
463 | unlock_dir(lower_dir_dentry); | 463 | unlock_dir(lower_dir_dentry); |
464 | dput(lower_new_dentry); | 464 | dput(lower_new_dentry); |
465 | dput(lower_old_dentry); | 465 | dput(lower_old_dentry); |
466 | d_drop(lower_old_dentry); | ||
467 | d_drop(new_dentry); | ||
468 | d_drop(old_dentry); | ||
469 | return rc; | 466 | return rc; |
470 | } | 467 | } |
471 | 468 | ||