diff options
| author | Mathias Krause <minipli@googlemail.com> | 2013-09-30 22:05:40 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-10-02 22:08:24 +0200 |
| commit | 6865d1e834be84ddd5808d93d5035b492346c64a (patch) | |
| tree | a970a98630b927756028377cb1d28b83535982be /net/unix/diag.c | |
| parent | Merge branch 'connector' (diff) | |
| download | linux-6865d1e834be84ddd5808d93d5035b492346c64a.tar.xz 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/diag.c')
| -rw-r--r-- | net/unix/diag.c | 1 |
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); |