Commit 5ea33af9 authored by Christian Göttsche's avatar Christian Göttsche Committed by Paul Moore
Browse files

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: default avatarChristian Göttsche <cgzones@googlemail.com>
Signed-off-by: default avatarPaul Moore <paul@paul-moore.com>
parent 3eb8eaf2

security/selinux/hooks.c

+0 −2
Original line number Diff line number Diff line
@@ -3284,8 +3284,6 @@ static void selinux_inode_post_setxattr(struct dentry *dentry, const char *name, 
	isec->sid = newsid;

	isec->initialized = LABEL_INITIALIZED;

	spin_unlock(&isec->lock);



	return;

}



static int selinux_inode_getxattr(struct dentry *dentry, const char *name)

security/selinux/ss/conditional.c

+0 −2
Original line number Diff line number Diff line
@@ -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

security/selinux/ss/ebitmap.c

+0 −1
Original line number Diff line number Diff line
@@ -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)

security/selinux/ss/mls.c

+0 −1
Original line number Diff line number Diff line
@@ -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)

security/selinux/ss/services.c

+0 −2
Original line number Diff line number Diff line
@@ -529,8 +529,6 @@ static void security_dump_masked_av(struct policydb *policydb, 
	/* release scontext/tcontext */

	kfree(tcontext_name);

	kfree(scontext_name);



	return;

}



/*