diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-04-09 10:07:19 +0800 |
|---|---|---|
| committer | Kamal Mostafa <kamal@canonical.com> | 2013-06-20 08:55:32 -0700 |
| commit | e3dd0d910b3741780b125437ae7be4a994d0cac8 (patch) | |
| tree | 32f73a5a37063755041dfa421de41980fb9f4ac3 /net | |
| parent | d5426c8193ffcab00dfc2c3df567f21d2aa03805 (diff) | |
netrom: fix invalid use of sizeof in nr_recvmsg()
commit c802d759623acbd6e1ee9fbdabae89159a513913 upstream.
sizeof() when applied to a pointer typed expression gives the size of the
pointer, not that of the pointed data.
Introduced by commit 3ce5ef(netrom: fix info leak via msg_name in nr_recvmsg)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/netrom/af_netrom.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netrom/af_netrom.c b/net/netrom/af_netrom.c index 14c106b49e99..f334fbde50e8 100644 --- a/net/netrom/af_netrom.c +++ b/net/netrom/af_netrom.c @@ -1177,7 +1177,7 @@ static int nr_recvmsg(struct kiocb *iocb, struct socket *sock, } if (sax != NULL) { - memset(sax, 0, sizeof(sax)); + memset(sax, 0, sizeof(*sax)); sax->sax25_family = AF_NETROM; skb_copy_from_linear_data_offset(skb, 7, sax->sax25_call.ax25_call, AX25_ADDR_LEN); |