Use uint16 for port numbers, misc edition

remoting/protocol/chromium_socket_factory.cc

Index: remoting/protocol/chromium_socket_factory.cc
diff --git a/remoting/protocol/chromium_socket_factory.cc b/remoting/protocol/chromium_socket_factory.cc
index 0cfa1ad706e3b6f7ca68da5229d5a3007589d966..6777937d93d7b103d9b6f14a7b72edd2bec77157 100644
--- a/remoting/protocol/chromium_socket_factory.cc
+++ b/remoting/protocol/chromium_socket_factory.cc
@@ -122,10 +122,10 @@ bool UdpPacketSocket::Init(const rtc::SocketAddress& local_address,
     return false;
   }
 
-  for (uint16 port = min_port; port <= max_port; ++port) {
+  for (uint32 port = min_port; port <= max_port; ++port) {

[Peter Kasting, 2014/11/12 23:53:55]

The old code was unsafe if a caller passed 65535 for |max_port|.

     socket_.reset(new net::UDPServerSocket(NULL, net::NetLog::Source()));
     int result = socket_->Listen(
-        net::IPEndPoint(local_endpoint.address(), port));
+        net::IPEndPoint(local_endpoint.address(), static_cast<uint16>(port)));
     if (result == net::OK) {
       break;
     } else {