diff options
| author | Mathias Krause <minipli@googlemail.com> | 2013-09-30 20:05:40 (GMT) |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-10-02 20:08:24 (GMT) |
| commit | 6865d1e834be84ddd5808d93d5035b492346c64a (patch) | |
| tree | a970a98630b927756028377cb1d28b83535982be /net/unix/diag.c | |
| parent | d022af27908fad7696888196fe9a787c8995ab47 (diff) | |
| download | linux-fsl-qoriq-6865d1e834be84ddd5808d93d5035b492346c64a.tar.xz | |
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 d591091..86fa0f3 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); |