| From 95eeb751dc6e68948dbfa955f69e2b03f0e4b242 Mon Sep 17 00:00:00 2001 |
| From: Thomas Gleixner <tglx@linutronix.de> |
| Date: Wed, 9 Dec 2009 14:35:10 +0100 |
| Subject: [PATCH] audit: Do not send uninitialized data for AUDIT_TTY_GET |
| |
| commit 4c9a0a35cd3c2984a3c58bc22c7214472d01c9c9 in tip. |
| |
| audit_receive_msg() sends uninitialized data for AUDIT_TTY_GET when |
| the task was not found. |
| |
| Send reply only when task was found. |
| |
| Signed-off-by: Thomas Gleixner <tglx@linutronix.de> |
| Cc: Al Viro <viro@zeniv.linux.org.uk> |
| Cc: Eric Paris <eparis@redhat.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| --- |
| kernel/audit.c | 6 ++++-- |
| 1 files changed, 4 insertions(+), 2 deletions(-) |
| |
| diff --git a/kernel/audit.c b/kernel/audit.c |
| index c71bd26..3d04a64 100644 |
| --- a/kernel/audit.c |
| +++ b/kernel/audit.c |
| @@ -892,8 +892,10 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) |
| spin_unlock_irq(&tsk->sighand->siglock); |
| } |
| read_unlock(&tasklist_lock); |
| - audit_send_reply(NETLINK_CB(skb).pid, seq, AUDIT_TTY_GET, 0, 0, |
| - &s, sizeof(s)); |
| + |
| + if (!err) |
| + audit_send_reply(NETLINK_CB(skb).pid, seq, |
| + AUDIT_TTY_GET, 0, 0, &s, sizeof(s)); |
| break; |
| } |
| case AUDIT_TTY_SET: { |
| -- |
| 1.7.0.4 |
| |