summaryrefslogtreecommitdiffstats
path: root/fs/hfsplus/xattr_user.c
diff options
context:
space:
mode:
authorFabian Frederick <fabf@skynet.be>2015-04-16 21:46:58 +0200
committerLinus Torvalds <torvalds@linux-foundation.org>2015-04-17 15:04:04 +0200
commita3cef4cd6886c755d2148739699751900b51a365 (patch)
tree49e364b4aaee42340108e3653d268f10b89529e0 /fs/hfsplus/xattr_user.c
parenthfsplus: add missing curly braces in hfsplus_delete_cat() (diff)
downloadlinux-a3cef4cd6886c755d2148739699751900b51a365.tar.xz
linux-a3cef4cd6886c755d2148739699751900b51a365.zip
fs/hfsplus: move xattr_name allocation in hfsplus_getxattr()
security/trusted/user/osx getxattr did the same xattr_name initialization. Move that operation in hfsplus_getxattr(). Tested with security/trusted/user getfattr/setfattr Signed-off-by: Fabian Frederick <fabf@skynet.be> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to '')
-rw-r--r--fs/hfsplus/xattr_user.c17
1 files changed, 2 insertions, 15 deletions
diff --git a/fs/hfsplus/xattr_user.c b/fs/hfsplus/xattr_user.c
index 2b625a538b64..f8860df28842 100644
--- a/fs/hfsplus/xattr_user.c
+++ b/fs/hfsplus/xattr_user.c
@@ -14,22 +14,9 @@
static int hfsplus_user_getxattr(struct dentry *dentry, const char *name,
void *buffer, size_t size, int type)
{
- char *xattr_name;
- int res;
-
- if (!strcmp(name, ""))
- return -EINVAL;
-
- xattr_name = kmalloc(NLS_MAX_CHARSET_SIZE * HFSPLUS_ATTR_MAX_STRLEN + 1,
- GFP_KERNEL);
- if (!xattr_name)
- return -ENOMEM;
- strcpy(xattr_name, XATTR_USER_PREFIX);
- strcpy(xattr_name + XATTR_USER_PREFIX_LEN, name);
- res = hfsplus_getxattr(dentry, xattr_name, buffer, size);
- kfree(xattr_name);
- return res;
+ return hfsplus_getxattr(dentry, name, buffer, size,
+ XATTR_USER_PREFIX, XATTR_USER_PREFIX_LEN);
}
static int hfsplus_user_setxattr(struct dentry *dentry, const char *name,