]> git-server-git.apps.pok.os.sepia.ceph.com Git - xfsprogs-dev.git/commit
libfrog: fix incorrect FS_IOC_FSSETXATTR argument to ioctl()
authorArkadiusz Miśkiewicz <arekm@maven.pl>
Fri, 5 Dec 2025 14:31:48 +0000 (15:31 +0100)
committerAndrey Albershteyn <aalbersh@kernel.org>
Tue, 9 Dec 2025 10:23:50 +0000 (11:23 +0100)
commit3147d1d643c8278f1b75dedf3a64037bece7da53
tree2bf35a5d59175c46e1b040095dfa5762ec000378
parenteb1e9586ee67398c9a3bf67f29ba4b232a253191
libfrog: fix incorrect FS_IOC_FSSETXATTR argument to ioctl()

xfsprogs 6.17.0 has broken project quota due to incorrect argument
passed to FS_IOC_FSSETXATTR ioctl(). Instead of passing struct fsxattr,
struct file_attr was passed.

# LC_ALL=C /usr/sbin/xfs_quota -x -c "project -s -p /home/xxx 389701" /home
Setting up project 389701 (path /home/xxx)...
xfs_quota: cannot set project on /home/xxx: Invalid argument
Processed 1 (/etc/projects and cmdline) paths for project 389701 with
recursion depth infinite (-1).

ioctl(5, FS_IOC_FSSETXATTR, {fsx_xflags=FS_XFLAG_PROJINHERIT|FS_XFLAG_HASATTR, fsx_extsize=0, fsx_projid=0, fsx_cowextsize=389701}) = -1 EINVAL (Invalid argument)

There seems to be a double mistake which hides the original ioctl()
argument bug on old kernel with xfsprogs built against it. The size of
fa_xflags was also wrong in xfsprogs's linux.h header. This way when
xfsprogs is compiled on newer kernel but used with older kernel this bug
uncovers.

Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Arkadiusz Miśkiewicz <arekm@maven.pl>
Signed-off-by: Andrey Albershteyn <aalbersh@kernel.org>
include/linux.h
libfrog/file_attr.c