what happened to the double_unlock?  I'm a little bit behind, but pretty
sure there was a double_unlock() at one point?

On Tue, 2005-11-22 at 16:54 +0900, [EMAIL PROTECTED] wrote:
> 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

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

Reply via email to