diff options
author | Cyrill Gorcunov <gorcunov@openvz.org> | 2009-10-29 05:59:18 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-10-29 06:00:06 -0400 |
commit | 38bfd8f5bec496e8e0db8849e01c99a33479418a (patch) | |
tree | 1db96f539d28c5d4fb2f062b522f91f2f2212c2a /net/ipv4 | |
parent | ed3f2e40f3d438f4a1ec0a898173116cb26f106a (diff) |
net,socket: introduce DECLARE_SOCKADDR helper to catch overflow at build time
proto_ops->getname implies copying protocol specific data
into storage unit (particulary to __kernel_sockaddr_storage).
So when we implement new protocol support we should keep such
a detail in mind (which is easy to forget about).
Lets introduce DECLARE_SOCKADDR helper which check if
storage unit is not overfowed at build time.
Eventually inet_getname is switched to use DECLARE_SOCKADDR
(to show example of usage).
Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/af_inet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/af_inet.c b/net/ipv4/af_inet.c index 04a14b1600ac..538e84d0bcba 100644 --- a/net/ipv4/af_inet.c +++ b/net/ipv4/af_inet.c | |||
@@ -685,7 +685,7 @@ int inet_getname(struct socket *sock, struct sockaddr *uaddr, | |||
685 | { | 685 | { |
686 | struct sock *sk = sock->sk; | 686 | struct sock *sk = sock->sk; |
687 | struct inet_sock *inet = inet_sk(sk); | 687 | struct inet_sock *inet = inet_sk(sk); |
688 | struct sockaddr_in *sin = (struct sockaddr_in *)uaddr; | 688 | DECLARE_SOCKADDR(struct sockaddr_in *, sin, uaddr); |
689 | 689 | ||
690 | sin->sin_family = AF_INET; | 690 | sin->sin_family = AF_INET; |
691 | if (peer) { | 691 | if (peer) { |