diff options
| author | Michal Orzel <michalorzel.eng@gmail.com> | 2022-03-31 19:33:57 +0200 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2022-04-04 15:38:53 -0400 |
| commit | 0a9876f36b08706d9954d8ccb42d0cd85f210333 (patch) | |
| tree | 6637caf41df61bcef322027eff4857006ad6fe54 /security/selinux/hooks.c | |
| parent | 3123109284176b1532874591f7c81f3837bbdc17 (diff) | |
selinux: Remove redundant assignments
Get rid of redundant assignments which end up in values not being
read either because they are overwritten or the function ends.
Reported by clang-tidy [deadcode.DeadStores]
Signed-off-by: Michal Orzel <michalorzel.eng@gmail.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/hooks.c')
| -rw-r--r-- | security/selinux/hooks.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index e9e959343de9..f792835b3fb6 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c @@ -6487,7 +6487,6 @@ static int selinux_setprocattr(const char *name, void *value, size_t size) goto abort_change; /* Only allow single threaded processes to change context */ - error = -EPERM; if (!current_is_single_threaded()) { error = security_bounded_transition(&selinux_state, tsec->sid, sid); |