Date: Mon, 20 Jul 2009 19:16:42 +0000 (UTC) From: Edward Tomasz Napierala <trasz@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs zfs_acl.c zfs_dir.c zfs_vnops.c Message-ID: <200907201916.n6KJGsW5086537@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
trasz 2009-07-20 19:16:42 UTC
FreeBSD src repository
Modified files:
sys/cddl/contrib/opensolaris/uts/common/fs/zfs zfs_acl.c
zfs_dir.c
zfs_vnops.c
Log:
SVN rev 195785 on 2009-07-20 19:16:42Z by trasz
Fix permission handling for extended attributes in ZFS. Without
this change, ZFS uses SunOS Alternate Data Streams semantics - each
EA has its own permissions, which are set at EA creation time
and - unlike SunOS - invisible to the user and impossible to change.
From the user point of view, it's just broken: sometimes access
is granted when it shouldn't be, sometimes it's denied when
it shouldn't be.
This patch makes it behave just like UFS, i.e. depend on current
file permissions. Also, it fixes returned error codes (ENOATTR
instead of ENOENT) and makes listextattr(2) return 0 instead
of EPERM where there is no EA directory (i.e. the file never had
any EA).
Reviewed by: pjd (idea, not actual code)
Approved by: re (kib)
Revision Changes Path
1.9 +10 -0 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c
1.6 +10 -2 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_dir.c
1.45 +31 -1 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200907201916.n6KJGsW5086537>