summaryrefslogtreecommitdiff
path: root/security/selinux
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-10-14 13:46:55 +0100
committerPaul Moore <paul@paul-moore.com>2017-10-16 18:34:25 -0400
commitadd24372141855b057bf53982824c5fe50898957 (patch)
tree71759987310723d0cbfd85a535d3c0a3a7e26ffd /security/selinux
parent4298555df5e5cb956549de5b01e4c77b1e4bc00a (diff)
selinux: remove redundant assignment to str
str is being assigned to an empty string but str is never being read after that, so the assignment is redundant and can be removed. Moving the declaration of str to a more localised block, cleans up clang warning: "Value stored to 'str' is never read" Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux')
-rw-r--r--security/selinux/hooks.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
index 2dd4dd6bdbc1..f21f1e0e6452 100644
--- a/security/selinux/hooks.c
+++ b/security/selinux/hooks.c
@@ -3176,18 +3176,17 @@ static int selinux_inode_setxattr(struct dentry *dentry, const char *name,
if (!has_cap_mac_admin(true)) {
struct audit_buffer *ab;
size_t audit_size;
- const char *str;
/* We strip a nul only if it is at the end, otherwise the
* context contains a nul and we should audit that */
if (value) {
- str = value;
+ const char *str = value;
+
if (str[size - 1] == '\0')
audit_size = size - 1;
else
audit_size = size;
} else {
- str = "";
audit_size = 0;
}
ab = audit_log_start(current->audit_context, GFP_ATOMIC, AUDIT_SELINUX_ERR);