From 655aedf605e8b5bc99976499ae5bdb8d11aa4a93 Mon Sep 17 00:00:00 2001 From: Anoop C S Date: Mon, 8 Jan 2024 17:15:26 +0530 Subject: [PATCH] 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 --- src/client/Client.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/src/client/Client.cc b/src/client/Client.cc index 195738efc87db..9479ebfa21702 100644 --- 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)) -- 2.47.3