lkml.org 
[lkml]   [2007]   [Aug]   [8]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
DateWed, 08 Aug 2007 19:16:26 +0200
FromAndreas Gruenbacher <>
Subject[RFC 04/10] Temporary struct vfs_lookup in file_permission
Create a temporary struct vfs_lookup in file_permission() instead of
passing a NULL value.

Signed-off-by: Andreas Gruenbacher <ag@bestbits.at>

---
 fs/namei.c |   11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -292,14 +292,15 @@ int vfs_permission(struct vfs_lookup *lo
  *
  * Used to check for read/write/execute permissions on an already opened
  * file.
- *
- * Note:
- *	Do not use this function in new code.  All access checks should
- *	be done using vfs_permission().
  */
 int file_permission(struct file *file, int mask)
 {
-	return permission(file->f_path.dentry->d_inode, mask, NULL);
+	struct vfs_lookup lookup;
+
+	lookup.path = file->f_path;
+	lookup.flags = 0;
+
+	return permission(file->f_path.dentry->d_inode, mask, &lookup);
 }
 
 /*
-
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/

\
 
 \ /
  Last update: 2007-08-08 19:23    [from the cache]
©2003-2008