From: Brian Behlendorf <behlendorf1@llnl.gov>
Date: Thu, 30 Jun 2011 16:38:52 +0000 (-0700)
Subject: Always allow non-user xattrs
X-Git-Tag: zfs-0.6.0-rc5~20
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=285226eff39b807c154748a75cef57ddff08a352;p=zfs

Always allow non-user xattrs

Under Linux you may only disable USER xattrs.  The SECURITY,
SYSTEM, and TRUSTED xattr namespaces must always be available
if xattrs are supported by the filesystem.  The enforcement
of USER xattrs is performed in the zpl_xattr_user_* handlers.

Under Solaris there is only a single xattr namespace which
is managed globally.
---

diff --git a/module/zfs/zfs_vnops.c b/module/zfs/zfs_vnops.c
index 828253f96..ae61b4304 100644
--- a/module/zfs/zfs_vnops.c
+++ b/module/zfs/zfs_vnops.c
@@ -1112,14 +1112,6 @@ zfs_lookup(struct inode *dip, char *nm, struct inode **ipp, int flags,
 	*ipp = NULL;
 
 	if (flags & LOOKUP_XATTR) {
-		/*
-		 * If the xattr property is off, refuse the lookup request.
-		 */
-		if (!(zsb->z_flags & ZSB_XATTR)) {
-			ZFS_EXIT(zsb);
-			return (EINVAL);
-		}
-
 		/*
 		 * We don't allow recursive attributes..
 		 * Maybe someday we will.