summaryrefslogtreecommitdiffstats
path: root/net/unix
diff options
context:
space:
mode:
authorMathias Krause <minipli@googlemail.com>2013-09-30 22:05:40 +0200
committerDavid S. Miller <davem@davemloft.net>2013-10-02 16:08:24 -0400
commit6865d1e834be84ddd5808d93d5035b492346c64a (patch)
treea970a98630b927756028377cb1d28b83535982be /net/unix
parentd022af27908fad7696888196fe9a787c8995ab47 (diff)
downloadlinux-6865d1e834be84ddd5808d93d5035b492346c64a.tar.gz
linux-6865d1e834be84ddd5808d93d5035b492346c64a.tar.bz2
linux-6865d1e834be84ddd5808d93d5035b492346c64a.zip
unix_diag: fix info leak
When filling the netlink message we miss to wipe the pad field, therefore leak one byte of heap memory to userland. Fix this by setting pad to 0. Signed-off-by: Mathias Krause <minipli@googlemail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/unix')
-rw-r--r--net/unix/diag.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/net/unix/diag.c b/net/unix/diag.c
index d591091603bf..86fa0f3b2caf 100644
--- a/net/unix/diag.c
+++ b/net/unix/diag.c
@@ -124,6 +124,7 @@ static int sk_diag_fill(struct sock *sk, struct sk_buff *skb, struct unix_diag_r
rep->udiag_family = AF_UNIX;
rep->udiag_type = sk->sk_type;
rep->udiag_state = sk->sk_state;
+ rep->pad = 0;
rep->udiag_ino = sk_ino;
sock_diag_save_cookie(sk, rep->udiag_cookie);