diff options
| author |   Christian Göttsche <cgzones@googlemail.com> | 2022-02-17 15:21:29 +0100 |
|---|---|---|
| committer |   Paul Moore <paul@paul-moore.com> | 2022-02-18 10:42:12 -0500 |
| commit | 5ea33af9d430cd1dbfada1b839e0d317ed77bfac (patch) | |
| tree | 63fb0d597bc11506bfaa5c00a1593ba5ca04bad2 /security/selinux/ss | |
| parent | security: implement sctp_assoc_established hook in selinux (diff) | |
| download | linux-dev-5ea33af9d430cd1dbfada1b839e0d317ed77bfac.tar.xz linux-dev-5ea33af9d430cd1dbfada1b839e0d317ed77bfac.zip | |
selinux: drop return statement at end of void functions
Those return statements at the end of a void function are redundant.
Reported by clang-tidy [readability-redundant-control-flow]
Signed-off-by: Christian Göttsche <cgzones@googlemail.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/ss')
| -rw-r--r-- | security/selinux/ss/conditional.c | 2 | ||||
| -rw-r--r-- | security/selinux/ss/ebitmap.c | 1 | ||||
| -rw-r--r-- | security/selinux/ss/mls.c | 1 | ||||
| -rw-r--r-- | security/selinux/ss/services.c | 2 |
4 files changed, 0 insertions, 6 deletions
diff --git a/security/selinux/ss/conditional.c b/security/selinux/ss/conditional.c index 2ec6e5cd25d9..c46c419af512 100644 --- a/security/selinux/ss/conditional.c +++ b/security/selinux/ss/conditional.c @@ -566,8 +566,6 @@ void cond_compute_xperms(struct avtab *ctab, struct avtab_key *key, if (node->key.specified & AVTAB_ENABLED) services_compute_xperms_decision(xpermd, node); } - return; - } /* Determine whether additional permissions are granted by the conditional * av table, and if so, add them to the result diff --git a/security/selinux/ss/ebitmap.c b/security/selinux/ss/ebitmap.c index 61fcbb8d0f88..abde349c8321 100644 --- a/security/selinux/ss/ebitmap.c +++ b/security/selinux/ss/ebitmap.c @@ -359,7 +359,6 @@ void ebitmap_destroy(struct ebitmap *e) e->highbit = 0; e->node = NULL; - return; } int ebitmap_read(struct ebitmap *e, void *fp) diff --git a/security/selinux/ss/mls.c b/security/selinux/ss/mls.c index 3f5fd124342c..99571b19d4a9 100644 --- a/security/selinux/ss/mls.c +++ b/security/selinux/ss/mls.c @@ -156,7 +156,6 @@ void mls_sid_to_context(struct policydb *p, } *scontext = scontextp; - return; } int mls_level_isvalid(struct policydb *p, struct mls_level *l) diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c index 2f8db93e53b2..6901dc07680d 100644 --- a/security/selinux/ss/services.c +++ b/security/selinux/ss/services.c @@ -529,8 +529,6 @@ out: /* release scontext/tcontext */ kfree(tcontext_name); kfree(scontext_name); - - return; } /* |