From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Aneesh Kumar K.V" Subject: [PATCH -V8 4/9] vfs: Allow handle based open on symlinks Date: Mon, 17 May 2010 11:03:47 +0530 Message-ID: <1274074432-29695-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> References: <1274074432-29695-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Cc: linux-fsdevel@vger.kernel.org, sfrench@us.ibm.com, philippe.deniel@CEA.FR, linux-kernel@vger.kernel.org, "Aneesh Kumar K.V" To: hch@infradead.org, viro@zeniv.linux.org.uk, adilger@sun.com, corbet@lwn.net, serue@us.ibm.com, neilb@suse.de Return-path: In-Reply-To: <1274074432-29695-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org The patch update may_open to allow handle based open on symlinks. The file handle based API use file descritor returned from open_by_handle_at to do different file system operations. To find the link target name we need to get a file descriptor on symlinks. Signed-off-by: Aneesh Kumar K.V --- fs/namei.c | 17 ++++++++++++++--- 1 files changed, 14 insertions(+), 3 deletions(-) diff --git a/fs/namei.c b/fs/namei.c index 1fe4ee9..6edb9af 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1421,7 +1421,7 @@ int vfs_create(struct inode *dir, struct dentry *dentry, int mode, return error; } -int may_open(struct path *path, int acc_mode, int flag) +static int __may_open(struct path *path, int acc_mode, int flag, int handle) { struct dentry *dentry = path->dentry; struct inode *inode = dentry->d_inode; @@ -1432,7 +1432,13 @@ int may_open(struct path *path, int acc_mode, int flag) switch (inode->i_mode & S_IFMT) { case S_IFLNK: - return -ELOOP; + /* + * For file handle based open we should allow + * open of symlink. + */ + if (!handle) + return -ELOOP; + break; case S_IFDIR: if (acc_mode & MAY_WRITE) return -EISDIR; @@ -1472,6 +1478,11 @@ int may_open(struct path *path, int acc_mode, int flag) return break_lease(inode, flag); } +int may_open(struct path *path, int acc_mode, int flag) +{ + return __may_open(path, acc_mode, flag, 0); +} + static int handle_truncate(struct path *path) { struct inode *inode = path->dentry->d_inode; @@ -1569,7 +1580,7 @@ struct file *finish_open_handle(struct path *path, if (error) goto exit; } - error = may_open(path, acc_mode, open_flag); + error = __may_open(path, acc_mode, open_flag, 1); if (error) { if (will_truncate) mnt_drop_write(path->mnt); -- 1.7.1.78.g212f0