Skip to content

L
linux
Commit 8bc56704 authored by James Morris's avatar James Morris Committed by Linus Torvalds
Browse files
Options

[PATCH] xattr consolidation v3 - ext2 

This patch converts ext2 xattr and acl code to the new generic xattr API.
Signed-off-by: default avatarJames Morris <jmorris@redhat.com>
Signed-off-by: default avatarStephen Smalley <sds@epoch.ncsc.mil>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 96686888
Hide whitespace changes
Inline Side-by-side
Showing with 116 additions and 316 deletions
fs/ext2/acl.c
View file @ 8bc56704
...@@ -393,27 +393,27 @@ ext2_acl_chmod(struct inode *inode) ...@@ -393,27 +393,27 @@ ext2_acl_chmod(struct inode *inode)
* Extended attribut handlers * Extended attribut handlers
*/ */
static size_t static size_t
ext2_xattr_list_acl_access(char *list, struct inode *inode, ext2_xattr_list_acl_access(struct inode *inode, char *list, size_t list_size,
const char *name, int name_len) const char *name, size_t name_len)
{ {
const size_t size = sizeof(XATTR_NAME_ACL_ACCESS); const size_t size = sizeof(XATTR_NAME_ACL_ACCESS);
if (!test_opt(inode->i_sb, POSIX_ACL)) if (!test_opt(inode->i_sb, POSIX_ACL))
return 0; return 0;
if (list) if (list && (size <= list_size))
memcpy(list, XATTR_NAME_ACL_ACCESS, size); memcpy(list, XATTR_NAME_ACL_ACCESS, size);
return size; return size;
} }
static size_t static size_t
ext2_xattr_list_acl_default(char *list, struct inode *inode, ext2_xattr_list_acl_default(struct inode *inode, char *list, size_t list_size,
const char *name, int name_len) const char *name, size_t name_len)
{ {
const size_t size = sizeof(XATTR_NAME_ACL_DEFAULT); const size_t size = sizeof(XATTR_NAME_ACL_DEFAULT);
if (!test_opt(inode->i_sb, POSIX_ACL)) if (!test_opt(inode->i_sb, POSIX_ACL))
return 0; return 0;
if (list) if (list && (size <= list_size))
memcpy(list, XATTR_NAME_ACL_DEFAULT, size); memcpy(list, XATTR_NAME_ACL_DEFAULT, size);
return size; return size;
} }
...@@ -505,45 +505,16 @@ ext2_xattr_set_acl_default(struct inode *inode, const char *name, ...@@ -505,45 +505,16 @@ ext2_xattr_set_acl_default(struct inode *inode, const char *name,
return ext2_xattr_set_acl(inode, ACL_TYPE_DEFAULT, value, size); return ext2_xattr_set_acl(inode, ACL_TYPE_DEFAULT, value, size);
} }
struct ext2_xattr_handler ext2_xattr_acl_access_handler = { struct xattr_handler ext2_xattr_acl_access_handler = {
.prefix = XATTR_NAME_ACL_ACCESS, .prefix = XATTR_NAME_ACL_ACCESS,
.list = ext2_xattr_list_acl_access, .list = ext2_xattr_list_acl_access,
.get = ext2_xattr_get_acl_access, .get = ext2_xattr_get_acl_access,
.set = ext2_xattr_set_acl_access, .set = ext2_xattr_set_acl_access,
}; };
struct ext2_xattr_handler ext2_xattr_acl_default_handler = { struct xattr_handler ext2_xattr_acl_default_handler = {
.prefix = XATTR_NAME_ACL_DEFAULT, .prefix = XATTR_NAME_ACL_DEFAULT,
.list = ext2_xattr_list_acl_default, .list = ext2_xattr_list_acl_default,
.get = ext2_xattr_get_acl_default, .get = ext2_xattr_get_acl_default,
.set = ext2_xattr_set_acl_default, .set = ext2_xattr_set_acl_default,
}; };
void
exit_ext2_acl(void)
{
ext2_xattr_unregister(EXT2_XATTR_INDEX_POSIX_ACL_ACCESS,
&ext2_xattr_acl_access_handler);
ext2_xattr_unregister(EXT2_XATTR_INDEX_POSIX_ACL_DEFAULT,
&ext2_xattr_acl_default_handler);
}
int __init
init_ext2_acl(void)
{
int error;
error = ext2_xattr_register(EXT2_XATTR_INDEX_POSIX_ACL_ACCESS,
&ext2_xattr_acl_access_handler);
if (error)
goto fail;
error = ext2_xattr_register(EXT2_XATTR_INDEX_POSIX_ACL_DEFAULT,
&ext2_xattr_acl_default_handler);
if (error)
goto fail;
return 0;
fail:
exit_ext2_acl();
return error;
}
fs/ext2/acl.h
View file @ 8bc56704
...@@ -63,9 +63,6 @@ extern int ext2_permission (struct inode *, int, struct nameidata *); ...@@ -63,9 +63,6 @@ extern int ext2_permission (struct inode *, int, struct nameidata *);
extern int ext2_acl_chmod (struct inode *); extern int ext2_acl_chmod (struct inode *);
extern int ext2_init_acl (struct inode *, struct inode *); extern int ext2_init_acl (struct inode *, struct inode *);
extern int init_ext2_acl(void);
extern void exit_ext2_acl(void);
#else #else
#include <linux/sched.h> #include <linux/sched.h>
#define ext2_permission NULL #define ext2_permission NULL
......
fs/ext2/file.c
View file @ 8bc56704
...@@ -57,10 +57,12 @@ struct file_operations ext2_file_operations = { ...@@ -57,10 +57,12 @@ struct file_operations ext2_file_operations = {
struct inode_operations ext2_file_inode_operations = { struct inode_operations ext2_file_inode_operations = {
.truncate = ext2_truncate, .truncate = ext2_truncate,
.setxattr = ext2_setxattr, #ifdef CONFIG_EXT2_FS_XATTR
.getxattr = ext2_getxattr, .setxattr = generic_setxattr,
.getxattr = generic_getxattr,
.listxattr = ext2_listxattr, .listxattr = ext2_listxattr,
.removexattr = ext2_removexattr, .removexattr = generic_removexattr,
#endif
.setattr = ext2_setattr, .setattr = ext2_setattr,
.permission = ext2_permission, .permission = ext2_permission,
}; };
fs/ext2/namei.c
View file @ 8bc56704
...@@ -395,19 +395,23 @@ struct inode_operations ext2_dir_inode_operations = { ...@@ -395,19 +395,23 @@ struct inode_operations ext2_dir_inode_operations = {
.rmdir = ext2_rmdir, .rmdir = ext2_rmdir,
.mknod = ext2_mknod, .mknod = ext2_mknod,
.rename = ext2_rename, .rename = ext2_rename,
.setxattr = ext2_setxattr, #ifdef CONFIG_EXT2_FS_XATTR
.getxattr = ext2_getxattr, .setxattr = generic_setxattr,
.getxattr = generic_getxattr,
.listxattr = ext2_listxattr, .listxattr = ext2_listxattr,
.removexattr = ext2_removexattr, .removexattr = generic_removexattr,
#endif
.setattr = ext2_setattr, .setattr = ext2_setattr,
.permission = ext2_permission, .permission = ext2_permission,
}; };
struct inode_operations ext2_special_inode_operations = { struct inode_operations ext2_special_inode_operations = {
.setxattr = ext2_setxattr, #ifdef CONFIG_EXT2_FS_XATTR
.getxattr = ext2_getxattr, .setxattr = generic_setxattr,
.getxattr = generic_getxattr,
.listxattr = ext2_listxattr, .listxattr = ext2_listxattr,
.removexattr = ext2_removexattr, .removexattr = generic_removexattr,
#endif
.setattr = ext2_setattr, .setattr = ext2_setattr,
.permission = ext2_permission, .permission = ext2_permission,
}; };
fs/ext2/super.c
View file @ 8bc56704
...@@ -800,6 +800,7 @@ static int ext2_fill_super(struct super_block *sb, void *data, int silent) ...@@ -800,6 +800,7 @@ static int ext2_fill_super(struct super_block *sb, void *data, int silent)
*/ */
sb->s_op = &ext2_sops; sb->s_op = &ext2_sops;
sb->s_export_op = &ext2_export_ops; sb->s_export_op = &ext2_export_ops;
sb->s_xattr = ext2_xattr_handlers;
root = iget(sb, EXT2_ROOT_INO); root = iget(sb, EXT2_ROOT_INO);
sb->s_root = d_alloc_root(root); sb->s_root = d_alloc_root(root);
if (!sb->s_root) { if (!sb->s_root) {
......
fs/ext2/symlink.c
View file @ 8bc56704
...@@ -32,17 +32,21 @@ struct inode_operations ext2_symlink_inode_operations = { ...@@ -32,17 +32,21 @@ struct inode_operations ext2_symlink_inode_operations = {
.readlink = generic_readlink, .readlink = generic_readlink,
.follow_link = page_follow_link_light, .follow_link = page_follow_link_light,
.put_link = page_put_link, .put_link = page_put_link,
.setxattr = ext2_setxattr, #ifdef CONFIG_EXT2_FS_XATTR
.getxattr = ext2_getxattr, .setxattr = generic_setxattr,
.getxattr = generic_getxattr,
.listxattr = ext2_listxattr, .listxattr = ext2_listxattr,
.removexattr = ext2_removexattr, .removexattr = generic_removexattr,
#endif
}; };
struct inode_operations ext2_fast_symlink_inode_operations = { struct inode_operations ext2_fast_symlink_inode_operations = {
.readlink = generic_readlink, .readlink = generic_readlink,
.follow_link = ext2_follow_link, .follow_link = ext2_follow_link,
.setxattr = ext2_setxattr, #ifdef CONFIG_EXT2_FS_XATTR
.getxattr = ext2_getxattr, .setxattr = generic_setxattr,
.getxattr = generic_getxattr,
.listxattr = ext2_listxattr, .listxattr = ext2_listxattr,
.removexattr = ext2_removexattr, .removexattr = generic_removexattr,
#endif
}; };
fs/ext2/xattr.c
View file @ 8bc56704
...@@ -6,6 +6,9 @@ ...@@ -6,6 +6,9 @@
* Fix by Harrison Xing <harrison@mountainviewdata.com>. * Fix by Harrison Xing <harrison@mountainviewdata.com>.
* Extended attributes for symlinks and special files added per * Extended attributes for symlinks and special files added per
* suggestion of Luka Renko <luka.renko@hermes.si>. * suggestion of Luka Renko <luka.renko@hermes.si>.
* xattr consolidation Copyright (c) 2004 James Morris <jmorris@redhat.com>,
* Red Hat Inc.
*
*/ */
/* /*
...@@ -62,8 +65,6 @@ ...@@ -62,8 +65,6 @@
#include "acl.h" #include "acl.h"
/* These symbols may be needed by a module. */ /* These symbols may be needed by a module. */
EXPORT_SYMBOL(ext2_xattr_register);
EXPORT_SYMBOL(ext2_xattr_unregister);
EXPORT_SYMBOL(ext2_xattr_get); EXPORT_SYMBOL(ext2_xattr_get);
EXPORT_SYMBOL(ext2_xattr_list); EXPORT_SYMBOL(ext2_xattr_list);
EXPORT_SYMBOL(ext2_xattr_set); EXPORT_SYMBOL(ext2_xattr_set);
...@@ -104,101 +105,40 @@ static void ext2_xattr_rehash(struct ext2_xattr_header *, ...@@ -104,101 +105,40 @@ static void ext2_xattr_rehash(struct ext2_xattr_header *,
struct ext2_xattr_entry *); struct ext2_xattr_entry *);
static struct mb_cache *ext2_xattr_cache; static struct mb_cache *ext2_xattr_cache;
static struct ext2_xattr_handler *ext2_xattr_handlers[EXT2_XATTR_INDEX_MAX];
static rwlock_t ext2_handler_lock = RW_LOCK_UNLOCKED;
int
ext2_xattr_register(int name_index, struct ext2_xattr_handler *handler)
{
int error = -EINVAL;
if (name_index > 0 && name_index <= EXT2_XATTR_INDEX_MAX) { static struct xattr_handler *ext2_xattr_handler_map[EXT2_XATTR_INDEX_MAX] = {
write_lock(&ext2_handler_lock); [EXT2_XATTR_INDEX_USER] = &ext2_xattr_user_handler,
if (!ext2_xattr_handlers[name_index-1]) { #ifdef CONFIG_EXT2_FS_POSIX_ACL
ext2_xattr_handlers[name_index-1] = handler; [EXT2_XATTR_INDEX_POSIX_ACL_ACCESS] = &ext2_xattr_acl_access_handler,
error = 0; [EXT2_XATTR_INDEX_POSIX_ACL_DEFAULT] = &ext2_xattr_acl_default_handler,
} #endif
write_unlock(&ext2_handler_lock); [EXT2_XATTR_INDEX_TRUSTED] = &ext2_xattr_trusted_handler,
} #ifdef CONFIG_EXT2_FS_SECURITY
return error; [EXT2_XATTR_INDEX_SECURITY] = &ext2_xattr_security_handler,
} #endif
};
void
ext2_xattr_unregister(int name_index, struct ext2_xattr_handler *handler)
{
if (name_index > 0 || name_index <= EXT2_XATTR_INDEX_MAX) {
write_lock(&ext2_handler_lock);
ext2_xattr_handlers[name_index-1] = NULL;
write_unlock(&ext2_handler_lock);
}
}
static inline const char *
strcmp_prefix(const char *a, const char *a_prefix)
{
while (*a_prefix && *a == *a_prefix) {
a++;
a_prefix++;
}
return *a_prefix ? NULL : a;
}
/* struct xattr_handler *ext2_xattr_handlers[] = {
* Decode the extended attribute name, and translate it into &ext2_xattr_user_handler,
* the name_index and name suffix. &ext2_xattr_trusted_handler,
*/ #ifdef CONFIG_EXT2_FS_POSIX_ACL
static struct ext2_xattr_handler * &ext2_xattr_acl_access_handler,
ext2_xattr_resolve_name(const char **name) &ext2_xattr_acl_default_handler,
{ #endif
struct ext2_xattr_handler *handler = NULL; #ifdef CONFIG_EXT2_FS_SECURITY
int i; &ext2_xattr_security_handler,
#endif
if (!*name) NULL
return NULL; };
read_lock(&ext2_handler_lock);
for (i=0; i<EXT2_XATTR_INDEX_MAX; i++) {
if (ext2_xattr_handlers[i]) {
const char *n = strcmp_prefix(*name,
ext2_xattr_handlers[i]->prefix);
if (n) {
handler = ext2_xattr_handlers[i];
*name = n;
break;
}
}
}
read_unlock(&ext2_handler_lock);
return handler;
}
static inline struct ext2_xattr_handler * static inline struct xattr_handler *
ext2_xattr_handler(int name_index) ext2_xattr_handler(int name_index)
{ {
struct ext2_xattr_handler *handler = NULL; struct xattr_handler *handler = NULL;
if (name_index > 0 && name_index <= EXT2_XATTR_INDEX_MAX) {
read_lock(&ext2_handler_lock);
handler = ext2_xattr_handlers[name_index-1];
read_unlock(&ext2_handler_lock);
}
return handler;
}
/* if (name_index > 0 && name_index <= EXT2_XATTR_INDEX_MAX)
* Inode operation getxattr() handler = ext2_xattr_handler_map[name_index];
* return handler;
* dentry->d_inode->i_sem: don't care
*/
ssize_t
ext2_getxattr(struct dentry *dentry, const char *name,
void *buffer, size_t size)
{
struct ext2_xattr_handler *handler;
struct inode *inode = dentry->d_inode;
handler = ext2_xattr_resolve_name(&name);
if (!handler)
return -EOPNOTSUPP;
return handler->get(inode, name, buffer, size);
} }
/* /*
...@@ -212,43 +152,6 @@ ext2_listxattr(struct dentry *dentry, char *buffer, size_t size) ...@@ -212,43 +152,6 @@ ext2_listxattr(struct dentry *dentry, char *buffer, size_t size)
return ext2_xattr_list(dentry->d_inode, buffer, size); return ext2_xattr_list(dentry->d_inode, buffer, size);
} }
/*
* Inode operation setxattr()
*
* dentry->d_inode->i_sem: down
*/
int
ext2_setxattr(struct dentry *dentry, const char *name,
const void *value, size_t size, int flags)
{
struct ext2_xattr_handler *handler;
struct inode *inode = dentry->d_inode;
if (size == 0)
value = ""; /* empty EA, do not remove */
handler = ext2_xattr_resolve_name(&name);
if (!handler)
return -EOPNOTSUPP;
return handler->set(inode, name, value, size, flags);
}
/*
* Inode operation removexattr()
*
* dentry->d_inode->i_sem: down
*/
int
ext2_removexattr(struct dentry *dentry, const char *name)
{
struct ext2_xattr_handler *handler;
struct inode *inode = dentry->d_inode;
handler = ext2_xattr_resolve_name(&name);
if (!handler)
return -EOPNOTSUPP;
return handler->set(inode, name, NULL, 0, XATTR_REPLACE);
}
/* /*
* ext2_xattr_get() * ext2_xattr_get()
* *
...@@ -367,7 +270,7 @@ ext2_xattr_list(struct inode *inode, char *buffer, size_t buffer_size) ...@@ -367,7 +270,7 @@ ext2_xattr_list(struct inode *inode, char *buffer, size_t buffer_size)
{ {
struct buffer_head *bh = NULL; struct buffer_head *bh = NULL;
struct ext2_xattr_entry *entry; struct ext2_xattr_entry *entry;
size_t size = 0; size_t total_size = 0;
char *buf, *end; char *buf, *end;
int error; int error;
...@@ -394,44 +297,37 @@ bad_block: ext2_error(inode->i_sb, "ext2_xattr_list", ...@@ -394,44 +297,37 @@ bad_block: ext2_error(inode->i_sb, "ext2_xattr_list",
error = -EIO; error = -EIO;
goto cleanup; goto cleanup;
} }
/* compute the size required for the list of attribute names */
for (entry = FIRST_ENTRY(bh); !IS_LAST_ENTRY(entry);
entry = EXT2_XATTR_NEXT(entry)) {
struct ext2_xattr_handler *handler;
struct ext2_xattr_entry *next =
EXT2_XATTR_NEXT(entry);
if ((char *)next >= end)
goto bad_block;
handler = ext2_xattr_handler(entry->e_name_index);
if (handler)
size += handler->list(NULL, inode, entry->e_name,
entry->e_name_len);
}
if (ext2_xattr_cache_insert(bh)) if (ext2_xattr_cache_insert(bh))
ea_idebug(inode, "cache insert failed"); ea_idebug(inode, "cache insert failed");
if (!buffer) {
error = size;
goto cleanup;
} else {
error = -ERANGE;
if (size > buffer_size)
goto cleanup;
}
/* list the attribute names */ /* list the attribute names */
buf = buffer; buf = buffer;
for (entry = FIRST_ENTRY(bh); !IS_LAST_ENTRY(entry); for (entry = FIRST_ENTRY(bh); !IS_LAST_ENTRY(entry);
entry = EXT2_XATTR_NEXT(entry)) { entry = EXT2_XATTR_NEXT(entry)) {
struct ext2_xattr_handler *handler; struct xattr_handler *handler;
struct ext2_xattr_entry *next = EXT2_XATTR_NEXT(entry);
if ((char *)next >= end)
goto bad_block;
handler = ext2_xattr_handler(entry->e_name_index); handler = ext2_xattr_handler(entry->e_name_index);
if (handler) if (handler) {
buf += handler->list(buf, inode, entry->e_name, size_t size = handler->list(inode, buf, buffer_size,
entry->e_name_len); entry->e_name,
entry->e_name_len);
if (buf) {
if (size > buffer_size) {
error = -ERANGE;
goto cleanup;
}
buf += size;
buffer_size -= size;
}
total_size += size;
}
} }
error = size; error = total_size;
cleanup: cleanup:
brelse(bh); brelse(bh);
...@@ -1120,66 +1016,16 @@ static void ext2_xattr_rehash(struct ext2_xattr_header *header, ...@@ -1120,66 +1016,16 @@ static void ext2_xattr_rehash(struct ext2_xattr_header *header,
int __init int __init
init_ext2_xattr(void) init_ext2_xattr(void)
{ {
int err;
err = ext2_xattr_register(EXT2_XATTR_INDEX_USER,
&ext2_xattr_user_handler);
if (err)
return err;
err = ext2_xattr_register(EXT2_XATTR_INDEX_TRUSTED,
&ext2_xattr_trusted_handler);
if (err)
goto out;
#ifdef CONFIG_EXT2_FS_SECURITY
err = ext2_xattr_register(EXT2_XATTR_INDEX_SECURITY,
&ext2_xattr_security_handler);
if (err)
goto out1;
#endif
#ifdef CONFIG_EXT2_FS_POSIX_ACL
err = init_ext2_acl();
if (err)
goto out2;
#endif
ext2_xattr_cache = mb_cache_create("ext2_xattr", NULL, ext2_xattr_cache = mb_cache_create("ext2_xattr", NULL,
sizeof(struct mb_cache_entry) + sizeof(struct mb_cache_entry) +
sizeof(struct mb_cache_entry_index), 1, 6); sizeof(struct mb_cache_entry_index), 1, 6);
if (!ext2_xattr_cache) { if (!ext2_xattr_cache)
err = -ENOMEM; return -ENOMEM;
goto out3;
}
return 0; return 0;
out3:
#ifdef CONFIG_EXT2_FS_POSIX_ACL
exit_ext2_acl();
out2:
#endif
#ifdef CONFIG_EXT2_FS_SECURITY
ext2_xattr_unregister(EXT2_XATTR_INDEX_SECURITY,
&ext2_xattr_security_handler);
out1:
#endif
ext2_xattr_unregister(EXT2_XATTR_INDEX_TRUSTED,
&ext2_xattr_trusted_handler);
out:
ext2_xattr_unregister(EXT2_XATTR_INDEX_USER,
&ext2_xattr_user_handler);
return err;
} }
void void
exit_ext2_xattr(void) exit_ext2_xattr(void)
{ {
mb_cache_destroy(ext2_xattr_cache); mb_cache_destroy(ext2_xattr_cache);
#ifdef CONFIG_EXT2_FS_POSIX_ACL
exit_ext2_acl();
#endif
#ifdef CONFIG_EXT2_FS_SECURITY
ext2_xattr_unregister(EXT2_XATTR_INDEX_SECURITY,
&ext2_xattr_security_handler);
#endif
ext2_xattr_unregister(EXT2_XATTR_INDEX_TRUSTED,
&ext2_xattr_trusted_handler);
ext2_xattr_unregister(EXT2_XATTR_INDEX_USER,
&ext2_xattr_user_handler);
} }
fs/ext2/xattr.h
View file @ 8bc56704
...@@ -57,23 +57,13 @@ struct ext2_xattr_entry { ...@@ -57,23 +57,13 @@ struct ext2_xattr_entry {
# ifdef CONFIG_EXT2_FS_XATTR # ifdef CONFIG_EXT2_FS_XATTR
struct ext2_xattr_handler { extern struct xattr_handler ext2_xattr_user_handler;
char *prefix; extern struct xattr_handler ext2_xattr_trusted_handler;
size_t (*list)(char *list, struct inode *inode, const char *name, extern struct xattr_handler ext2_xattr_acl_access_handler;
int name_len); extern struct xattr_handler ext2_xattr_acl_default_handler;
int (*get)(struct inode *inode, const char *name, void *buffer, extern struct xattr_handler ext2_xattr_security_handler;
size_t size);
int (*set)(struct inode *inode, const char *name, const void *buffer,
size_t size, int flags);
};
extern int ext2_xattr_register(int, struct ext2_xattr_handler *);
extern void ext2_xattr_unregister(int, struct ext2_xattr_handler *);
extern int ext2_setxattr(struct dentry *, const char *, const void *, size_t, int);
extern ssize_t ext2_getxattr(struct dentry *, const char *, void *, size_t);
extern ssize_t ext2_listxattr(struct dentry *, char *, size_t); extern ssize_t ext2_listxattr(struct dentry *, char *, size_t);
extern int ext2_removexattr(struct dentry *, const char *);
extern int ext2_xattr_get(struct inode *, int, const char *, void *, size_t); extern int ext2_xattr_get(struct inode *, int, const char *, void *, size_t);
extern int ext2_xattr_list(struct inode *, char *, size_t); extern int ext2_xattr_list(struct inode *, char *, size_t);
...@@ -85,11 +75,9 @@ extern void ext2_xattr_put_super(struct super_block *); ...@@ -85,11 +75,9 @@ extern void ext2_xattr_put_super(struct super_block *);
extern int init_ext2_xattr(void); extern int init_ext2_xattr(void);
extern void exit_ext2_xattr(void); extern void exit_ext2_xattr(void);
extern struct xattr_handler *ext2_xattr_handlers[];
# else /* CONFIG_EXT2_FS_XATTR */ # else /* CONFIG_EXT2_FS_XATTR */
# define ext2_setxattr NULL
# define ext2_getxattr NULL
# define ext2_listxattr NULL
# define ext2_removexattr NULL
static inline int static inline int
ext2_xattr_get(struct inode *inode, int name_index, ext2_xattr_get(struct inode *inode, int name_index,
...@@ -132,9 +120,7 @@ exit_ext2_xattr(void) ...@@ -132,9 +120,7 @@ exit_ext2_xattr(void)
{ {
} }
# endif /* CONFIG_EXT2_FS_XATTR */ #define ext2_xattr_handlers NULL
extern struct ext2_xattr_handler ext2_xattr_user_handler; # endif /* CONFIG_EXT2_FS_XATTR */
extern struct ext2_xattr_handler ext2_xattr_trusted_handler;
extern struct ext2_xattr_handler ext2_xattr_security_handler;
fs/ext2/xattr_security.c
View file @ 8bc56704
...@@ -11,17 +11,18 @@ ...@@ -11,17 +11,18 @@
#include "xattr.h" #include "xattr.h"
static size_t static size_t
ext2_xattr_security_list(char *list, struct inode *inode, ext2_xattr_security_list(struct inode *inode, char *list, size_t list_size,
const char *name, int name_len) const char *name, size_t name_len)
{ {
const int prefix_len = sizeof(XATTR_SECURITY_PREFIX)-1; const int prefix_len = sizeof(XATTR_SECURITY_PREFIX)-1;
const size_t total_len = prefix_len + name_len + 1;
if (list) { if (list && (total_len <= list_size)) {
memcpy(list, XATTR_SECURITY_PREFIX, prefix_len); memcpy(list, XATTR_SECURITY_PREFIX, prefix_len);
memcpy(list+prefix_len, name, name_len); memcpy(list+prefix_len, name, name_len);
list[prefix_len + name_len] = '\0'; list[prefix_len + name_len] = '\0';
} }
return prefix_len + name_len + 1; return total_len;
} }
static int static int
...@@ -44,7 +45,7 @@ ext2_xattr_security_set(struct inode *inode, const char *name, ...@@ -44,7 +45,7 @@ ext2_xattr_security_set(struct inode *inode, const char *name,
value, size, flags); value, size, flags);
} }
struct ext2_xattr_handler ext2_xattr_security_handler = { struct xattr_handler ext2_xattr_security_handler = {
.prefix = XATTR_SECURITY_PREFIX, .prefix = XATTR_SECURITY_PREFIX,
.list = ext2_xattr_security_list, .list = ext2_xattr_security_list,
.get = ext2_xattr_security_get, .get = ext2_xattr_security_get,
......
fs/ext2/xattr_trusted.c
View file @ 8bc56704
...@@ -15,20 +15,21 @@ ...@@ -15,20 +15,21 @@
#define XATTR_TRUSTED_PREFIX "trusted." #define XATTR_TRUSTED_PREFIX "trusted."
static size_t static size_t
ext2_xattr_trusted_list(char *list, struct inode *inode, ext2_xattr_trusted_list(struct inode *inode, char *list, size_t list_size,
const char *name, int name_len) const char *name, size_t name_len)
{ {
const int prefix_len = sizeof(XATTR_TRUSTED_PREFIX)-1; const int prefix_len = sizeof(XATTR_TRUSTED_PREFIX)-1;
const size_t total_len = prefix_len + name_len + 1;
if (!capable(CAP_SYS_ADMIN)) if (!capable(CAP_SYS_ADMIN))
return 0; return 0;
if (list) { if (list && (total_len <= list_size)) {
memcpy(list, XATTR_TRUSTED_PREFIX, prefix_len); memcpy(list, XATTR_TRUSTED_PREFIX, prefix_len);
memcpy(list+prefix_len, name, name_len); memcpy(list+prefix_len, name, name_len);
list[prefix_len + name_len] = '\0'; list[prefix_len + name_len] = '\0';
} }
return prefix_len + name_len + 1; return total_len;
} }
static int static int
...@@ -55,7 +56,7 @@ ext2_xattr_trusted_set(struct inode *inode, const char *name, ...@@ -55,7 +56,7 @@ ext2_xattr_trusted_set(struct inode *inode, const char *name,
value, size, flags); value, size, flags);
} }
struct ext2_xattr_handler ext2_xattr_trusted_handler = { struct xattr_handler ext2_xattr_trusted_handler = {
.prefix = XATTR_TRUSTED_PREFIX, .prefix = XATTR_TRUSTED_PREFIX,
.list = ext2_xattr_trusted_list, .list = ext2_xattr_trusted_list,
.get = ext2_xattr_trusted_get, .get = ext2_xattr_trusted_get,
......
fs/ext2/xattr_user.c
View file @ 8bc56704
...@@ -14,20 +14,21 @@ ...@@ -14,20 +14,21 @@
#define XATTR_USER_PREFIX "user." #define XATTR_USER_PREFIX "user."
static size_t static size_t
ext2_xattr_user_list(char *list, struct inode *inode, ext2_xattr_user_list(struct inode *inode, char *list, size_t list_size,
const char *name, int name_len) const char *name, size_t name_len)
{ {
const int prefix_len = sizeof(XATTR_USER_PREFIX)-1; const size_t prefix_len = sizeof(XATTR_USER_PREFIX)-1;
const size_t total_len = prefix_len + name_len + 1;
if (!test_opt(inode->i_sb, XATTR_USER)) if (!test_opt(inode->i_sb, XATTR_USER))
return 0; return 0;
if (list) { if (list && (total_len <= list_size)) {
memcpy(list, XATTR_USER_PREFIX, prefix_len); memcpy(list, XATTR_USER_PREFIX, prefix_len);
memcpy(list+prefix_len, name, name_len); memcpy(list+prefix_len, name, name_len);
list[prefix_len + name_len] = '\0'; list[prefix_len + name_len] = '\0';
} }
return prefix_len + name_len + 1; return total_len;
} }
static int static int
...@@ -68,23 +69,9 @@ ext2_xattr_user_set(struct inode *inode, const char *name, ...@@ -68,23 +69,9 @@ ext2_xattr_user_set(struct inode *inode, const char *name,
value, size, flags); value, size, flags);
} }
struct ext2_xattr_handler ext2_xattr_user_handler = { struct xattr_handler ext2_xattr_user_handler = {
.prefix = XATTR_USER_PREFIX, .prefix = XATTR_USER_PREFIX,
.list = ext2_xattr_user_list, .list = ext2_xattr_user_list,
.get = ext2_xattr_user_get, .get = ext2_xattr_user_get,
.set = ext2_xattr_user_set, .set = ext2_xattr_user_set,
}; };
int __init
init_ext2_xattr_user(void)
{
return ext2_xattr_register(EXT2_XATTR_INDEX_USER,
&ext2_xattr_user_handler);
}
void
exit_ext2_xattr_user(void)
{
ext2_xattr_unregister(EXT2_XATTR_INDEX_USER,
&ext2_xattr_user_handler);
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7