From: Mimi Zohar <zohar@linux.vnet.ibm.com>
To: linux-kernel@vger.kernel.org
Cc: Mimi Zohar <zohar@linux.vnet.ibm.com>,
linux-security-module@vger.kernel.org,
linux-fsdevel@vger.kernel.org, James Morris <jmorris@namei.org>,
David Safford <safford@watson.ibm.com>,
Dave Hansen <dave@linux.vnet.ibm.com>,
Mimi Zohar <zohar@us.ibm.com>
Subject: [PATCH 03/15] xattr: define vfs_getxattr_alloc and vfs_xattr_cmp
Date: Thu, 24 Jun 2010 14:10:19 -0400 [thread overview]
Message-ID: <1277403031-3080-4-git-send-email-zohar@linux.vnet.ibm.com> (raw)
In-Reply-To: <1277403031-3080-1-git-send-email-zohar@linux.vnet.ibm.com>
vfs_getxattr_alloc() and vfs_xattr_cmp() are two new kernel xattr
helper functions. vfs_getxattr_alloc() first allocates memory for
the requested xattr and then retrieves it. vfs_xattr_cmp() compares
a given value with the contents of an extended attribute.
Signed-off-by: Mimi Zohar <zohar@us.ibm.com>
Acked-by: Serge E. Hallyn <serue@us.ibm.com>
---
fs/xattr.c | 58 +++++++++++++++++++++++++++++++++++++++++++++++++
include/linux/xattr.h | 5 +++-
2 files changed, 62 insertions(+), 1 deletions(-)
diff --git a/fs/xattr.c b/fs/xattr.c
index 01bb813..e1f6b8b 100644
--- a/fs/xattr.c
+++ b/fs/xattr.c
@@ -159,6 +159,64 @@ out_noalloc:
}
EXPORT_SYMBOL_GPL(xattr_getsecurity);
+/*
+ * vfs_getxattr_alloc - allocate memory, if necessary, before calling getxattr
+ *
+ * Allocate memory, if not already allocated, or re-allocate correct size,
+ * before retrieving the extended attribute.
+ *
+ * Returns the result of alloc, if failed, or the getxattr operation.
+ */
+ssize_t
+vfs_getxattr_alloc(struct dentry *dentry, const char *name, char **xattr_value,
+ size_t xattr_size, gfp_t flags)
+{
+ struct inode *inode = dentry->d_inode;
+ char *value = *xattr_value;
+ int error;
+
+ error = xattr_permission(inode, name, MAY_READ);
+ if (error)
+ return error;
+
+ if (!inode->i_op->getxattr)
+ return -EOPNOTSUPP;
+
+ error = inode->i_op->getxattr(dentry, name, NULL, 0);
+ if (error < 0)
+ return error;
+
+ if (!value || (error > xattr_size)) {
+ value = krealloc(*xattr_value, error + 1, flags);
+ if (!value)
+ return -ENOMEM;
+ memset(value, 0, error + 1);
+ }
+
+ error = inode->i_op->getxattr(dentry, name, value, error);
+ *xattr_value = value;
+ return error;
+}
+
+/* Compare an extended attribute value with the given value */
+int vfs_xattr_cmp(struct dentry *dentry, const char *xattr_name,
+ const char *value, size_t size, gfp_t flags)
+{
+ char *xattr_value = NULL;
+ int rc;
+
+ rc = vfs_getxattr_alloc(dentry, xattr_name, &xattr_value, 0, flags);
+ if (rc < 0)
+ return rc;
+
+ if ((rc != size) || (memcmp(xattr_value, value, rc) != 0))
+ rc = -EINVAL;
+ else
+ rc = 0;
+ kfree(xattr_value);
+ return rc;
+}
+
ssize_t
vfs_getxattr(struct dentry *dentry, const char *name, void *value, size_t size)
{
diff --git a/include/linux/xattr.h b/include/linux/xattr.h
index 62ca853..f921a4f 100644
--- a/include/linux/xattr.h
+++ b/include/linux/xattr.h
@@ -68,7 +68,10 @@ ssize_t generic_getxattr(struct dentry *dentry, const char *name, void *buffer,
ssize_t generic_listxattr(struct dentry *dentry, char *buffer, size_t buffer_size);
int generic_setxattr(struct dentry *dentry, const char *name, const void *value, size_t size, int flags);
int generic_removexattr(struct dentry *dentry, const char *name);
-
+ssize_t vfs_getxattr_alloc(struct dentry *dentry, const char *name,
+ char **xattr_value, size_t size, gfp_t flags);
+int vfs_xattr_cmp(struct dentry *dentry, const char *xattr_name,
+ const char *value, size_t size, gfp_t flags);
#endif /* __KERNEL__ */
#endif /* _LINUX_XATTR_H */
--
1.6.6.1
next prev parent reply other threads:[~2010-06-24 18:10 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-24 18:10 [PATCH 00/15] EVM Mimi Zohar
2010-06-24 18:10 ` [PATCH 01/15] integrity: move ima inode integrity data management Mimi Zohar
2010-06-24 18:10 ` [PATCH 02/15] security: move LSM xattrnames to xattr.h Mimi Zohar
2010-06-25 3:49 ` Casey Schaufler
2010-06-25 11:15 ` Mimi Zohar
2010-06-27 16:54 ` Casey Schaufler
2010-06-28 1:35 ` Mimi Zohar
2010-06-24 18:10 ` Mimi Zohar [this message]
2010-06-24 18:10 ` [PATCH 04/15] evm: re-release Mimi Zohar
2010-06-24 18:10 ` [PATCH 05/15] ima: move ima_file_free before releasing the file Mimi Zohar
2010-06-24 18:10 ` [PATCH 06/15] security: imbed evm calls in security hooks Mimi Zohar
2010-06-24 18:10 ` [PATCH 07/15] evm: inode post removexattr Mimi Zohar
2010-06-24 18:10 ` [PATCH 08/15] evm: imbed evm_inode_post_setattr Mimi Zohar
2010-06-24 18:10 ` [PATCH 09/15] evm: inode_post_init Mimi Zohar
2010-06-24 18:10 ` [PATCH 10/15] fs: add evm_inode_post_init calls Mimi Zohar
2010-06-24 18:10 ` [PATCH 11/15] ima: integrity appraisal extension Mimi Zohar
2010-06-24 18:10 ` [PATCH 12/15] ima: appraise default rules Mimi Zohar
2010-06-24 18:10 ` [PATCH 13/15] ima: inode post_setattr Mimi Zohar
2010-06-24 18:10 ` [PATCH 14/15] ima: add ima_inode_setxattr and ima_inode_removexattr Mimi Zohar
2010-06-24 18:10 ` [PATCH 15/15] ima: appraise measurement required Mimi Zohar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1277403031-3080-4-git-send-email-zohar@linux.vnet.ibm.com \
--to=zohar@linux.vnet.ibm.com \
--cc=dave@linux.vnet.ibm.com \
--cc=jmorris@namei.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-security-module@vger.kernel.org \
--cc=safford@watson.ibm.com \
--cc=zohar@us.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).