diff options
| author | Carmeli Tamir | 2019-07-06 17:07:38 +0200 |
|---|---|---|
| committer | James Morris | 2019-07-07 04:55:54 +0200 |
| commit | c5eaab1d131d0a6272df7d55a971a67400d63f56 (patch) | |
| tree | 52ff65f3ec93ff57884e7a4ed9846df87c28c4e9 /security | |
| parent | security: Make capability_hooks static (diff) | |
| download | kernel-qcow2-linux-c5eaab1d131d0a6272df7d55a971a67400d63f56.tar.gz kernel-qcow2-linux-c5eaab1d131d0a6272df7d55a971a67400d63f56.tar.xz kernel-qcow2-linux-c5eaab1d131d0a6272df7d55a971a67400d63f56.zip | |
security/commoncap: Use xattr security prefix len
Using the existing defined XATTR_SECURITY_PREFIX_LEN instead of
sizeof(XATTR_SECURITY_PREFIX) - 1. Pretty simple cleanup.
Signed-off-by: Carmeli Tamir <carmeli.tamir@gmail.com>
Signed-off-by: James Morris <jamorris@linux.microsoft.com>
Diffstat (limited to 'security')
| -rw-r--r-- | security/commoncap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/commoncap.c b/security/commoncap.c index afd9679ca866..1156178a0c52 100644 --- a/security/commoncap.c +++ b/security/commoncap.c @@ -920,7 +920,7 @@ int cap_inode_setxattr(struct dentry *dentry, const char *name, /* Ignore non-security xattrs */ if (strncmp(name, XATTR_SECURITY_PREFIX, - sizeof(XATTR_SECURITY_PREFIX) - 1) != 0) + XATTR_SECURITY_PREFIX_LEN) != 0) return 0; /* @@ -952,7 +952,7 @@ int cap_inode_removexattr(struct dentry *dentry, const char *name) /* Ignore non-security xattrs */ if (strncmp(name, XATTR_SECURITY_PREFIX, - sizeof(XATTR_SECURITY_PREFIX) - 1) != 0) + XATTR_SECURITY_PREFIX_LEN) != 0) return 0; if (strcmp(name, XATTR_NAME_CAPS) == 0) { |