lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 8 Aug 2013 16:46:54 +0200
From: Miklos Szeredi <miklos@...redi.hu>
To: Anand Avati <avati@...hat.com>
Cc: Ric Wheeler <rwheeler@...hat.com>,
Al Viro <viro@...iv.linux.org.uk>,
Brian Foster <bfoster@...hat.com>,
David Howells <dhowells@...hat.com>,
Eric Paris <eparis@...hat.com>,
Linux-Fsdevel <linux-fsdevel@...r.kernel.org>,
Kernel Mailing List <linux-kernel@...r.kernel.org>,
Linux NFS list <linux-nfs@...r.kernel.org>,
Trond Myklebust <Trond.Myklebust@...app.com>,
Steven Whitehouse <swhiteho@...hat.com>,
"mszeredi@...e.cz" <mszeredi@...e.cz>
Subject: Re: [PATCH 4/4] fuse: drop dentry on failed revalidate
On Wed, Aug 7, 2013 at 6:31 PM, Anand Avati <avati@...hat.com> wrote:
> Thanks for attaching the thread. Was very educative! I still do not quite
> understand - will umount() still work when inode_permission()[->getattr()]
> on the ancestors fail (with ESTALE etc.)?
Several cases:
1) mountpoint was removed
2) mountpoint's ancestor removed
3) mountpoint or ancestor moved
1) and 3) would only cause revalidate to fail, not inode operations
(permission or getattr) since the inodes of ancestors still exist.
2) could error out if the ->permission got to the server. I'm not
sure it will and I don't have an NFS setup handy to test.
So it doesn't necessarily work but it can be made to work, and as
Linus pointed out, it might still be the least worst option.
BTW, attaching a little fuse filesystem that I've been testing this
with. It's read-only, but the point here is that the tree is modified
*externally*, so that's not an issue.
Thanks,
Miklos
View attachment "fuse_lo.c" of type "text/x-csrc" (8227 bytes)
Powered by blists - more mailing lists