client: Fix parsing of xattr namespace in removexattr

To match the kernel ceph client, as mentioned by the inline comments,
remove the comparison with "system." xattr namespace in removexattr()
code path. This is in preparation to handle the case where POSIX ACL
xattrs are treated differently by kernel during removexattr operation.

ref: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/fs/ceph/xattr.c#n21

Signed-off-by: Anoop C S <anoopcs@cryptolab.net>

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 195738efc87db5724dbbd50e6d3e602bf79ba190..9479ebfa21702f6ee64c473df06de7f1ee6ec704 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -14069,7 +14069,6 @@ int Client::_removexattr(Inode *in, const char *name, const UserPerm& perms)
 
   // same xattrs supported by kernel client
   if (strncmp(name, "user.", 5) &&
-      strncmp(name, "system.", 7) &&
       strncmp(name, "security.", 9) &&
       strncmp(name, "trusted.", 8) &&
       strncmp(name, "ceph.", 5))