Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(328)

Unified Diff: dart/runtime/bin/socket_win.cc

Issue 60293003: Version 0.8.10.5 (Closed) Base URL: http://dart.googlecode.com/svn/trunk/
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dart/runtime/bin/socket_patch.dart ('k') | dart/runtime/vm/intermediate_language.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/runtime/bin/socket_win.cc
===================================================================
--- dart/runtime/bin/socket_win.cc (revision 29908)
+++ dart/runtime/bin/socket_win.cc (working copy)
@@ -85,7 +85,7 @@
}
-bool Socket::GetRemotePeer(intptr_t fd, char *host, intptr_t *port) {
+SocketAddress* Socket::GetRemotePeer(intptr_t fd, intptr_t* port) {
ASSERT(reinterpret_cast<Handle*>(fd)->is_socket());
SocketHandle* socket_handle = reinterpret_cast<SocketHandle*>(fd);
RawAddr raw;
@@ -94,23 +94,13 @@
&raw.addr,
&size)) {
Log::PrintErr("Error getpeername: %d\n", WSAGetLastError());
- return false;
+ return NULL;
}
*port = SocketAddress::GetAddrPort(&raw);
// Clear the port before calling WSAAddressToString as WSAAddressToString
// includes the port in the formatted string.
SocketAddress::SetAddrPort(&raw, 0);
- DWORD len = INET6_ADDRSTRLEN;
- int err = WSAAddressToStringA(&raw.addr,
- sizeof(raw),
- NULL,
- host,
- &len);
- if (err != 0) {
- Log::PrintErr("Error WSAAddressToString: %d\n", WSAGetLastError());
- return false;
- }
- return true;
+ return new SocketAddress(&raw.addr);
}
« no previous file with comments | « dart/runtime/bin/socket_patch.dart ('k') | dart/runtime/vm/intermediate_language.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698