From owner-cvs-src-old@FreeBSD.ORG Mon Jul 20 19:16:55 2009 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 2C15B10657EA for ; Mon, 20 Jul 2009 19:16:54 +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 9A69B8FC12 for ; Mon, 20 Jul 2009 19:16:54 +0000 (UTC) (envelope-from trasz@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n6KJGsXw086538 for ; Mon, 20 Jul 2009 19:16:54 GMT (envelope-from trasz@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n6KJGsW5086537 for cvs-src-old@freebsd.org; Mon, 20 Jul 2009 19:16:54 GMT (envelope-from trasz@repoman.freebsd.org) Message-Id: <200907201916.n6KJGsW5086537@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to trasz@repoman.freebsd.org using -f From: Edward Tomasz Napierala Date: Mon, 20 Jul 2009 19:16:42 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD 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: Mon, 20 Jul 2009 19:16:55 -0000 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