diff options
| author | Andreas Gruenbacher <agruenba@redhat.com> | 2018-09-18 06:36:36 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2018-09-18 13:30:48 +0200 |
| commit | ffc4c92227db5699493e43eb140b4cb5904c30ff (patch) | |
| tree | b77ac1265e074070094902511e74c9033de16bfb /crypto | |
| parent | Linux 4.19-rc1 (diff) | |
| download | linux-ffc4c92227db5699493e43eb140b4cb5904c30ff.tar.xz linux-ffc4c92227db5699493e43eb140b4cb5904c30ff.zip | |
sysfs: Do not return POSIX ACL xattrs via listxattr
Commit 786534b92f3c introduced a regression that caused listxattr to
return the POSIX ACL attribute names even though sysfs doesn't support
POSIX ACLs. This happens because simple_xattr_list checks for NULL
i_acl / i_default_acl, but inode_init_always initializes those fields
to ACL_NOT_CACHED ((void *)-1). For example:
$ getfattr -m- -d /sys
/sys: system.posix_acl_access: Operation not supported
/sys: system.posix_acl_default: Operation not supported
Fix this in simple_xattr_list by checking if the filesystem supports POSIX ACLs.
Fixes: 786534b92f3c ("tmpfs: listxattr should include POSIX ACL xattrs")
Reported-by: Marc Aurèle La France <tsi@tuyoix.net>
Tested-by: Marc Aurèle La France <tsi@tuyoix.net>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Cc: stable@vger.kernel.org # v4.5+
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'crypto')
0 files changed, 0 insertions, 0 deletions