Date: Fri, 22 Jan 2010 11:42:44 +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: <201001221142.o0MBgvdD035409@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
trasz 2010-01-22 11:42:44 UTC
FreeBSD src repository
Modified files: (Branch: RELENG_7)
sys/cddl/contrib/opensolaris/uts/common/fs/zfs zfs_acl.c
zfs_dir.c
zfs_vnops.c
Log:
SVN rev 202805 on 2010-01-22 11:42:44Z by trasz
MFC c195785:
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).
Tested by: cperciva
Revision Changes Path
1.3.2.4 +10 -0 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c
1.3.2.3 +10 -2 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_dir.c
1.31.2.11 +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?201001221142.o0MBgvdD035409>