diff options
author | Paul Moore <pmoore@redhat.com> | 2015-11-04 08:23:52 -0500 |
---|---|---|
committer | Paul Moore <pmoore@redhat.com> | 2015-11-04 08:23:52 -0500 |
commit | 233a68667cf4c134d07ef7e22bdd77786b5c7360 (patch) | |
tree | 963ef28d17205ff33ac59818fe08fb5b813d75a4 /kernel/audit.c | |
parent | c5ea6efda6ff0fd591d6b7a2e1ba086b196dd864 (diff) |
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) | |||
701 | return err; | 701 | return err; |
702 | } | 702 | } |
703 | 703 | ||
704 | static int audit_log_common_recv_msg(struct audit_buffer **ab, u16 msg_type) | 704 | static void audit_log_common_recv_msg(struct audit_buffer **ab, u16 msg_type) |
705 | { | 705 | { |
706 | uid_t uid = from_kuid(&init_user_ns, current_uid()); | 706 | uid_t uid = from_kuid(&init_user_ns, current_uid()); |
707 | pid_t pid = task_tgid_nr(current); | 707 | pid_t pid = task_tgid_nr(current); |
708 | 708 | ||
709 | if (!audit_enabled && msg_type != AUDIT_USER_AVC) { | 709 | if (!audit_enabled && msg_type != AUDIT_USER_AVC) { |
710 | *ab = NULL; | 710 | *ab = NULL; |
711 | return 0; | 711 | return; |
712 | } | 712 | } |
713 | 713 | ||
714 | *ab = audit_log_start(NULL, GFP_KERNEL, msg_type); | 714 | *ab = audit_log_start(NULL, GFP_KERNEL, msg_type); |
715 | if (unlikely(!*ab)) | 715 | if (unlikely(!*ab)) |
716 | return 0; | 716 | return; |
717 | audit_log_format(*ab, "pid=%d uid=%u", pid, uid); | 717 | audit_log_format(*ab, "pid=%d uid=%u", pid, uid); |
718 | audit_log_session_info(*ab); | 718 | audit_log_session_info(*ab); |
719 | audit_log_task_context(*ab); | 719 | audit_log_task_context(*ab); |
720 | |||
721 | return 0; | ||
722 | } | 720 | } |
723 | 721 | ||
724 | int is_audit_feature_set(int i) | 722 | int is_audit_feature_set(int i) |