Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 5 May 2010 09:01:16 +0000 (UTC)
From:      Dmitry Morozovsky <marck@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-7@freebsd.org
Subject:   svn commit: r207653 - stable/7/sys/cddl/contrib/opensolaris/uts/common/fs/zfs
Message-ID:  <201005050901.o4591Ghe001189@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: marck (doc committer)
Date: Wed May  5 09:01:15 2010
New Revision: 207653
URL: http://svn.freebsd.org/changeset/base/207653

Log:
  MFC  r207068:
  
    Allow to modify directory's content even if the ZFS_NOUNLINK (SF_NOUNLINK,
    sunlnk) flag is set. We only deny dirctory's removal or rename.
  
    PR:           kern/143343
    Approved by:	pjd

Modified:
  stable/7/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c
Directory Properties:
  stable/7/sys/   (props changed)
  stable/7/sys/cddl/contrib/opensolaris/   (props changed)
  stable/7/sys/contrib/dev/acpica/   (props changed)
  stable/7/sys/contrib/pf/   (props changed)

Modified: stable/7/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c
==============================================================================
--- stable/7/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c	Wed May  5 08:58:58 2010	(r207652)
+++ stable/7/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_acl.c	Wed May  5 09:01:15 2010	(r207653)
@@ -2227,11 +2227,24 @@ zfs_zaccess_common(znode_t *zp, uint32_t
 		return (EPERM);
 	}
 
+#ifdef sun
 	if ((v4_mode & (ACE_DELETE | ACE_DELETE_CHILD)) &&
 	    (zp->z_phys->zp_flags & ZFS_NOUNLINK)) {
 		*check_privs = B_FALSE;
 		return (EPERM);
 	}
+#else
+	/*
+	 * In FreeBSD we allow to modify directory's content is ZFS_NOUNLINK
+	 * (sunlnk) is set. We just don't allow directory removal, which is
+	 * handled in zfs_zaccess_delete().
+	 */
+	if ((v4_mode & ACE_DELETE) &&
+	    (zp->z_phys->zp_flags & ZFS_NOUNLINK)) {
+		*check_privs = B_FALSE;
+		return (EPERM);
+	}
+#endif
 
 	if (((v4_mode & (ACE_READ_DATA|ACE_EXECUTE)) &&
 	    (zp->z_phys->zp_flags & ZFS_AV_QUARANTINED))) {



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201005050901.o4591Ghe001189>