linux-kernel - Re: [patch 6/9] fuse: add export operations

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080509194030.GA29118@infradead.org>
Date:	Fri, 9 May 2008 15:40:30 -0400
From:	Christoph Hellwig <hch@...radead.org>
To:	Miklos Szeredi <miklos@...redi.hu>
Cc:	akpm@...ux-foundation.org, linux-fsdevel@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [patch 6/9] fuse: add export operations

> +struct fuse_inode_handle
> +{
> +	u64 nodeid;
> +	u32 generation;
> +};

This is the same filehandle type XFS uses for 64bit inode filesystems
where the parent is not encoded.  I'll post a patch soon to move that
64bit inode handling to common code.  I don't think it's a good idea
to delay your patch until that happens, but can you chose the fid
types to be the same as XFS does currently so that a conversion is
easily possible.  The fh_type should be 0x81 (fh type 1 order by 0x80
to imply it's 64bit inodes)

In case you want to look at that patch I'v uploaded the current
version at http://verein.lst.de/~hch/generic-64bit-inode-export

> +static struct dentry *fuse_get_dentry(struct super_block *sb,
> +				      struct fuse_inode_handle *handle)
> +{
> +	struct inode *inode;
> +	struct dentry *entry;
> +	int err = -ESTALE;
> +
> +	if (handle->nodeid == 0)
> +		goto out_err;
> +
> +	inode = ilookup5(sb, handle->nodeid, fuse_inode_eq, &handle->nodeid);
> +	if (!inode)
> +		goto out_err;

Only finding in-memory inodes for nfs export is quite dangerous.  I
think it would be a much better idea to support whatever upcalls are
needed to find the object if it's not in memory.  Without that the nfs
exporting support is hardly useable.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/