diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-04-09 10:07:19 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-05-01 08:56:40 -0700 |
| commit | aad877b259293436b2bdfb1005c9bb29bb17cce5 (patch) | |
| tree | b7c65afd89d92b17b10b9142258fe356909457e8 /net | |
| parent | dc85f620e827440f520551b2ff6222bc92b00fa9 (diff) | |
| download | lwn-aad877b259293436b2bdfb1005c9bb29bb17cce5.tar.gz lwn-aad877b259293436b2bdfb1005c9bb29bb17cce5.zip | |
netrom: fix invalid use of sizeof in nr_recvmsg()
[ Upstream commit c802d759623acbd6e1ee9fbdabae89159a513913 ]
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: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
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 aca28b9d2ada..3df7c5a4ac8e 100644 --- a/net/netrom/af_netrom.c +++ b/net/netrom/af_netrom.c @@ -1178,7 +1178,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); |