diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-04-22 16:24:14 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-04-25 01:47:58 -0400 |
commit | d9152097acc5f9fdc0bf1a8e3454634dfae47742 (patch) | |
tree | 7afcef4aac17bdfaeea49501fe5e5ef622fe5aa4 /drivers/isdn | |
parent | 7a3b68434b1b5fb7b9a6184efb26822cd1a54cc8 (diff) |
isdn: mISDN: set ->family in ->getname()
The "maddr->family" variable was not set but instead it leaked stack
information to userspace.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/isdn')
-rw-r--r-- | drivers/isdn/mISDN/socket.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/isdn/mISDN/socket.c b/drivers/isdn/mISDN/socket.c index 8b07f83d48ad..e47dcb9d1e91 100644 --- a/drivers/isdn/mISDN/socket.c +++ b/drivers/isdn/mISDN/socket.c | |||
@@ -578,6 +578,7 @@ data_sock_getname(struct socket *sock, struct sockaddr *addr, | |||
578 | lock_sock(sk); | 578 | lock_sock(sk); |
579 | 579 | ||
580 | *addr_len = sizeof(*maddr); | 580 | *addr_len = sizeof(*maddr); |
581 | maddr->family = AF_ISDN; | ||
581 | maddr->dev = _pms(sk)->dev->id; | 582 | maddr->dev = _pms(sk)->dev->id; |
582 | maddr->channel = _pms(sk)->ch.nr; | 583 | maddr->channel = _pms(sk)->ch.nr; |
583 | maddr->sapi = _pms(sk)->ch.addr & 0xff; | 584 | maddr->sapi = _pms(sk)->ch.addr & 0xff; |