From owner-cvs-src-old@FreeBSD.ORG Fri Jan 22 11:42:57 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A9FEA10656A7 for ; Fri, 22 Jan 2010 11:42:57 +0000 (UTC) (envelope-from trasz@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 9674C8FC1C for ; Fri, 22 Jan 2010 11:42:57 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o0MBgvYR035410 for ; Fri, 22 Jan 2010 11:42:57 GMT (envelope-from trasz@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o0MBgvdD035409 for cvs-src-old@freebsd.org; Fri, 22 Jan 2010 11:42:57 GMT (envelope-from trasz@repoman.freebsd.org) Message-Id: <201001221142.o0MBgvdD035409@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to trasz@repoman.freebsd.org using -f From: Edward Tomasz Napierala Date: Fri, 22 Jan 2010 11:42:44 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_7 Subject: cvs commit: src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs zfs_acl.c zfs_dir.c zfs_vnops.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 Jan 2010 11:42:57 -0000 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