Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758138Ab0FORNf (ORCPT ); Tue, 15 Jun 2010 13:13:35 -0400 Received: from e23smtp06.au.ibm.com ([202.81.31.148]:49040 "EHLO e23smtp06.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758023Ab0FORNc (ORCPT ); Tue, 15 Jun 2010 13:13:32 -0400 From: "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, hooanon05@yahoo.co.jp, bfields@fieldses.org Cc: linux-fsdevel@vger.kernel.org, sfrench@us.ibm.com, philippe.deniel@CEA.FR, linux-kernel@vger.kernel.org, "Aneesh Kumar K.V" Subject: [PATCH -V14 04/11] vfs: Allow handle based open on symlinks Date: Tue, 15 Jun 2010 22:42:54 +0530 Message-Id: <1276621981-2774-5-git-send-email-aneesh.kumar@linux.vnet.ibm.com> X-Mailer: git-send-email 1.7.1.331.ga5080 In-Reply-To: <1276621981-2774-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> References: <1276621981-2774-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2061 Lines: 67 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 c2d19c7..417bf53 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1422,7 +1422,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; @@ -1433,7 +1433,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; @@ -1473,6 +1479,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; @@ -1570,7 +1581,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.331.ga5080 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/