I am afraid this is a bug.
Forgetting dput() can make system unreliable, since the non-exsisting
file (or dir) may be misunderstood as if it exists.


Junjiro Okajima


diff -u -p -r1.1 inode.c
--- unionfs-20051118-1120/inode.c       20 Nov 2005 04:06:50 -0000      1.1
+++ unionfs-20051118-1120/inode.c       22 Nov 2005 06:35:40 -0000
@@ -107,6 +107,8 @@ static int unionfs_create(struct inode *
 
                /* This will dput our double gotten parent. */
                unlock_rename(hidden_dir_dentry, hidden_dir_dentry);
+               DPUT(hidden_dir_dentry);
+               DPUT(hidden_dir_dentry);
 
                DPUT(new_hidden_dentry);
 

_______________________________________________
unionfs mailing list
[email protected]
http://www.fsl.cs.sunysb.edu/mailman/listinfo/unionfs

Reply via email to