[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Dazuko-devel] DazukoFS 3.1.4-rc1 posted

From: John Ogness
Subject: Re: [Dazuko-devel] DazukoFS 3.1.4-rc1 posted
Date: Tue, 19 Oct 2010 14:20:38 +0200
User-agent: Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux)

On 2010-10-19, Lino Sanfilippo <address@hidden> wrote:
>> IMPORTANT: This new version fixes an inode leak when removing
>> directories.
> I assume the fix is
>    dentry->d_inode->i_nlink = get_lower_inode(dentry->d_inode)->i_nlink;
> in dazukofs_rmdir(), right?


> I could not find anything similar in ecryptfs code. Is this also a
> bug in ecryptfs?

I assume that ecryptfs also has this problem. But I did not take the
time to verify this.

> I looked into the vfs code by i could not find out what exactly the
> effect of the line above is.  A few words to clarify this would be
> great.

The bug (and fix) were reported on the Dazuko Savannah page:


There you will also see a simple procedure to verify the bug. This
could be done very quickly and easily to verify that ecryptfs also has
the problem.

When you look at other filesystems, you see that they indeed decrement
the i_nlink count by 2 when rmdir is called. Rather than performing
this ourself, we just copy the count from the lower filesystem. Since
ecryptfs does not do this, I expect ecryptfs to also have an inode
leak with rmdir.

John Ogness

Dazuko Maintainer

reply via email to

[Prev in Thread] Current Thread [Next in Thread]