diff options
| author | David Howells <dhowells@redhat.com> | 2008-03-06 03:53:55 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-03-06 03:53:55 +0100 |
| commit | 1ff82fe0024e8070c38346b8abc1ff09612dea4c (patch) | |
| tree | 68f3eca9e9ce6d4ae4e38575ab1370fb3b911d76 | |
| parent | net/enc28j60: oops fix (diff) | |
| download | linux-1ff82fe0024e8070c38346b8abc1ff09612dea4c.tar.xz linux-1ff82fe0024e8070c38346b8abc1ff09612dea4c.zip | |
RxRPC: fix rxrpc_recvmsg()'s returning of msg_name
Fix rxrpc_recvmsg() to return msg_name correctly. We shouldn't
overwrite the *msg struct, but should rather write into msg->msg_name
(there's a '&' unary operator that shouldn't be there).
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/rxrpc/ar-recvmsg.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/rxrpc/ar-recvmsg.c b/net/rxrpc/ar-recvmsg.c index f19121d4795b..a39bf97f8830 100644 --- a/net/rxrpc/ar-recvmsg.c +++ b/net/rxrpc/ar-recvmsg.c @@ -143,7 +143,8 @@ int rxrpc_recvmsg(struct kiocb *iocb, struct socket *sock, /* copy the peer address and timestamp */ if (!continue_call) { if (msg->msg_name && msg->msg_namelen > 0) - memcpy(&msg->msg_name, &call->conn->trans->peer->srx, + memcpy(msg->msg_name, + &call->conn->trans->peer->srx, sizeof(call->conn->trans->peer->srx)); sock_recv_timestamp(msg, &rx->sk, skb); } |