xfs: remove the kuid/kgid conversion wrappers
Remove the XFS wrappers for converting from and to the kuid/kgid types. Mostly this means switching to VFS i_{u,g}id_{read,write} helpers, but in a few spots the calls to the conversion functions is open coded. To match the use of sb->s_user_ns in the helpers and other file systems, sb->s_user_ns is also used in the quota code. The ACL code already does the conversion in a grotty layering violation in the VFS xattr code, so it keeps using init_user_ns for the identity mapping. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
This commit is contained in:

committed by
Darrick J. Wong

parent
542951592c
commit
ba8adad5d0
@@ -67,10 +67,12 @@ xfs_acl_from_disk(
|
||||
|
||||
switch (acl_e->e_tag) {
|
||||
case ACL_USER:
|
||||
acl_e->e_uid = xfs_uid_to_kuid(be32_to_cpu(ace->ae_id));
|
||||
acl_e->e_uid = make_kuid(&init_user_ns,
|
||||
be32_to_cpu(ace->ae_id));
|
||||
break;
|
||||
case ACL_GROUP:
|
||||
acl_e->e_gid = xfs_gid_to_kgid(be32_to_cpu(ace->ae_id));
|
||||
acl_e->e_gid = make_kgid(&init_user_ns,
|
||||
be32_to_cpu(ace->ae_id));
|
||||
break;
|
||||
case ACL_USER_OBJ:
|
||||
case ACL_GROUP_OBJ:
|
||||
@@ -103,10 +105,12 @@ xfs_acl_to_disk(struct xfs_acl *aclp, const struct posix_acl *acl)
|
||||
ace->ae_tag = cpu_to_be32(acl_e->e_tag);
|
||||
switch (acl_e->e_tag) {
|
||||
case ACL_USER:
|
||||
ace->ae_id = cpu_to_be32(xfs_kuid_to_uid(acl_e->e_uid));
|
||||
ace->ae_id = cpu_to_be32(
|
||||
from_kuid(&init_user_ns, acl_e->e_uid));
|
||||
break;
|
||||
case ACL_GROUP:
|
||||
ace->ae_id = cpu_to_be32(xfs_kgid_to_gid(acl_e->e_gid));
|
||||
ace->ae_id = cpu_to_be32(
|
||||
from_kgid(&init_user_ns, acl_e->e_gid));
|
||||
break;
|
||||
default:
|
||||
ace->ae_id = cpu_to_be32(ACL_UNDEFINED_ID);
|
||||
|
Reference in New Issue
Block a user