diff options
| author | Mathias Krause <minipli@googlemail.com> | 2013-09-30 22:04:24 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-04 10:50:32 -0800 |
| commit | 4e0c56ab133fcb29fa3709c095f9347a033662c1 (patch) | |
| tree | 4ddbc8a55d63d790669e6319b037870b4166005e /kernel | |
| parent | 42979968049cc6a8ed569ae8ceb486e0637c7ce7 (diff) | |
| download | lwn-4e0c56ab133fcb29fa3709c095f9347a033662c1.tar.gz lwn-4e0c56ab133fcb29fa3709c095f9347a033662c1.zip | |
audit: fix info leak in AUDIT_GET requests
commit 64fbff9ae0a0a843365d922e0057fc785f23f0e3 upstream.
We leak 4 bytes of kernel stack in response to an AUDIT_GET request as
we miss to initialize the mask member of status_set. Fix that.
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Eric Paris <eparis@redhat.com>
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/audit.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/audit.c b/kernel/audit.c index a57194b61418..4478f7b2e9fd 100644 --- a/kernel/audit.c +++ b/kernel/audit.c @@ -684,6 +684,7 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) switch (msg_type) { case AUDIT_GET: + status_set.mask = 0; status_set.enabled = audit_enabled; status_set.failure = audit_failure; status_set.pid = audit_pid; |