Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

This project is archived. Its data is read-only.

Commit 6f1b5d7a authored Nov 04, 2015 by Yaowei Bai Committed by Paul Moore Nov 04, 2015

audit: audit_tree_match can be boolean



This patch makes audit_tree_match return bool to improve readability
due to this particular function only using either one or zero as its
return value.

No functional change.

Signed-off-by: Yaowei Bai <bywxiaobai@163.com>
[PM: tweaked the subject line]
Signed-off-by: Paul Moore <pmoore@redhat.com>

parent 9fcf836b

kernel/audit.h

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -301,7 +301,7 @@ extern int audit_exe_compare(struct task_struct *tsk, struct audit_fsnotify_mark
	#ifdef CONFIG_AUDIT_TREE		#ifdef CONFIG_AUDIT_TREE
	extern struct audit_chunk audit_tree_lookup(const struct inode );		extern struct audit_chunk audit_tree_lookup(const struct inode );
	extern void audit_put_chunk(struct audit_chunk *);		extern void audit_put_chunk(struct audit_chunk *);
	extern int audit_tree_match(struct audit_chunk , struct audit_tree );		extern bool audit_tree_match(struct audit_chunk , struct audit_tree );
	extern int audit_make_tree(struct audit_krule , char , u32);		extern int audit_make_tree(struct audit_krule , char , u32);
	extern int audit_add_tree_rule(struct audit_krule *);		extern int audit_add_tree_rule(struct audit_krule *);
	extern int audit_remove_tree_rule(struct audit_krule *);		extern int audit_remove_tree_rule(struct audit_krule *);

kernel/audit_tree.c

+3 −3

Original line number	Original line	Diff line number	Diff line
	@@ -197,13 +197,13 @@ struct audit_chunk audit_tree_lookup(const struct inode inode)
	return NULL;		return NULL;
	}		}

	int audit_tree_match(struct audit_chunk chunk, struct audit_tree tree)		bool audit_tree_match(struct audit_chunk chunk, struct audit_tree tree)
	{		{
	int n;		int n;
	for (n = 0; n < chunk->count; n++)		for (n = 0; n < chunk->count; n++)
	if (chunk->owners[n].owner == tree)		if (chunk->owners[n].owner == tree)
	return 1;		return true;
	return 0;		return false;
	}		}

	/* tagging and untagging inodes with trees */		/* tagging and untagging inodes with trees */