Gets the peer address (if available) in server requests.

net/socket/stream_listen_socket.cc

Index: net/socket/stream_listen_socket.cc
diff --git a/net/socket/stream_listen_socket.cc b/net/socket/stream_listen_socket.cc
index 960991b7c6da0be28862155cccf74cea0e56ffb6..e12631f18959342cfc277521fae430f46e8d67d5 100644
--- a/net/socket/stream_listen_socket.cc
+++ b/net/socket/stream_listen_socket.cc
@@ -100,6 +100,17 @@ int StreamListenSocket::GetLocalAddress(IPEndPoint* address) {
   return OK;
 }
 
+int StreamListenSocket::GetPeerAddress(IPEndPoint* address) {
+  SockaddrStorage storage;
+  if (getpeername(socket_, storage.addr, &storage.addr_len)) {
+    return MapSystemError(errno);

[Ryan Sleevi, 2014/03/11 01:32:43]

|errno| won't be set on Win - it would be WSAGetLastError()

[gunsch, 2014/03/17 16:57:15]

Done.

+  } else if (!address->FromSockAddr(storage.addr, storage.addr_len)) {
+    return ERR_FAILED;

[Ryan Sleevi, 2014/03/11 01:32:43]

ERR_ADDRESS_INVALID

[gunsch, 2014/03/17 16:57:15]

Hm, why ERR_ADDRESS_INVALID? In GetLocalAddress above, ERR_FAILED is used here,
and same in UDBSocketLibEvent::GetPeerAddress.

[Ryan Sleevi, 2014/03/17 19:47:22]

Matches the TCPClientSocket.

I suspect the UDPClientSocket/this class are the ones lagging.

+  } else {
+    return OK;

[Ryan Sleevi, 2014/03/11 01:32:43]

don't use else after return statements

if (getpeername(socket_, storage.addr, &storage.addr_len)))
  return MapSystemError(errno);  // BUGGED as above

if (!address->FromSockAddr(storage.addr, storage.addr_len))
  return ERR_ADDRESS_INVALID;

return OK;

(see other examples of calls to getpeername() in net/

[gunsch, 2014/03/17 16:57:15]

Done.

+  }
+}
+
 SocketDescriptor StreamListenSocket::AcceptSocket() {
   SocketDescriptor conn = HANDLE_EINTR(accept(socket_, NULL, NULL));
   if (conn == kInvalidSocket)