tmpfs: simplify constructing "security.foo" xattr names

Use kasprintf() instead of doing it with kmalloc() + 2 x memcpy().

Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
Link: https://patch.msgid.link/20260605135322.2632068-3-mszeredi@redhat.com
Tested-by: Calum Mackay <calum.mackay@oracle.com>
Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>
This commit is contained in:
Miklos Szeredi 2026-06-05 15:53:17 +02:00 committed by Christian Brauner
parent 6a07814ff6
commit 832f4de4c8
No known key found for this signature in database
GPG Key ID: 91C61BC06578DCA2

View File

@ -4232,7 +4232,6 @@ static int shmem_initxattrs(struct inode *inode,
struct shmem_sb_info *sbinfo = SHMEM_SB(inode->i_sb);
const struct xattr *xattr;
size_t ispace = 0;
size_t len;
CLASS(simple_xattrs, xattrs)();
if (IS_ERR(xattrs))
@ -4260,17 +4259,11 @@ static int shmem_initxattrs(struct inode *inode,
if (IS_ERR(new_xattr))
break;
len = strlen(xattr->name) + 1;
new_xattr->name = kmalloc(XATTR_SECURITY_PREFIX_LEN + len,
GFP_KERNEL_ACCOUNT);
new_xattr->name = kasprintf(GFP_KERNEL_ACCOUNT,
XATTR_SECURITY_PREFIX "%s", xattr->name);
if (!new_xattr->name)
break;
memcpy(new_xattr->name, XATTR_SECURITY_PREFIX,
XATTR_SECURITY_PREFIX_LEN);
memcpy(new_xattr->name + XATTR_SECURITY_PREFIX_LEN,
xattr->name, len);
if (simple_xattr_add(xattrs, new_xattr))
break;
retain_and_null_ptr(new_xattr);