diff options
author | Paul Moore <paul@paul-moore.com> | 2020-03-05 14:55:43 -0500 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2020-03-05 14:55:43 -0500 |
commit | 5e729e111eaf37b7941c678cb84af62539a4799a (patch) | |
tree | 560da6da64809b755af8b5e5a320d482b3ab26be /security/selinux/ss/conditional.c | |
parent | 34a2dab488bcaf2ac2198d7b305794280d73207b (diff) | |
download | linux-5e729e111eaf37b7941c678cb84af62539a4799a.tar.gz linux-5e729e111eaf37b7941c678cb84af62539a4799a.tar.bz2 linux-5e729e111eaf37b7941c678cb84af62539a4799a.zip |
selinux: avtab_init() and cond_policydb_init() return void
The avtab_init() and cond_policydb_init() functions always return
zero so mark them as returning void and update the callers not to
check for a return value.
Suggested-by: Stephen Smalley <stephen.smalley.work@gmail.com>
Reviewed-by: Ondrej Mosnacek <omosnace@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/ss/conditional.c')
-rw-r--r-- | security/selinux/ss/conditional.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/security/selinux/ss/conditional.c b/security/selinux/ss/conditional.c index cce4a75fb3e7..939a74fd8fb4 100644 --- a/security/selinux/ss/conditional.c +++ b/security/selinux/ss/conditional.c @@ -125,19 +125,13 @@ void evaluate_cond_nodes(struct policydb *p) evaluate_cond_node(p, &p->cond_list[i]); } -int cond_policydb_init(struct policydb *p) +void cond_policydb_init(struct policydb *p) { - int rc; - p->bool_val_to_struct = NULL; p->cond_list = NULL; p->cond_list_len = 0; - rc = avtab_init(&p->te_cond_avtab); - if (rc) - return rc; - - return 0; + avtab_init(&p->te_cond_avtab); } static void cond_node_destroy(struct cond_node *node) |