diff options
| author | Paul Moore <pmoore@redhat.com> | 2015-11-04 14:23:52 +0100 |
|---|---|---|
| committer | Paul Moore <pmoore@redhat.com> | 2015-11-04 14:23:52 +0100 |
| commit | 233a68667cf4c134d07ef7e22bdd77786b5c7360 (patch) | |
| tree | 963ef28d17205ff33ac59818fe08fb5b813d75a4 /kernel/audit.c | |
| parent | audit: removing unused variable (diff) | |
| download | linux-233a68667cf4c134d07ef7e22bdd77786b5c7360.tar.xz linux-233a68667cf4c134d07ef7e22bdd77786b5c7360.zip | |
audit: make audit_log_common_recv_msg() a void function
It always returns zero and no one is checking the return value.
Signed-off-by: Paul Moore <pmoore@redhat.com>
Diffstat (limited to 'kernel/audit.c')
| -rw-r--r-- | kernel/audit.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/kernel/audit.c b/kernel/audit.c index 85570f348ccf..8a056a32ded7 100644 --- a/kernel/audit.c +++ b/kernel/audit.c @@ -701,24 +701,22 @@ static int audit_netlink_ok(struct sk_buff *skb, u16 msg_type) return err; } -static int audit_log_common_recv_msg(struct audit_buffer **ab, u16 msg_type) +static void audit_log_common_recv_msg(struct audit_buffer **ab, u16 msg_type) { uid_t uid = from_kuid(&init_user_ns, current_uid()); pid_t pid = task_tgid_nr(current); if (!audit_enabled && msg_type != AUDIT_USER_AVC) { *ab = NULL; - return 0; + return; } *ab = audit_log_start(NULL, GFP_KERNEL, msg_type); if (unlikely(!*ab)) - return 0; + return; audit_log_format(*ab, "pid=%d uid=%u", pid, uid); audit_log_session_info(*ab); audit_log_task_context(*ab); - - return 0; } int is_audit_feature_set(int i) |