Correctly handle SSL Client Authentication requests when connecting...

remoting/jingle_glue/ssl_socket_adapter.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/jingle_glue/ssl_socket_adapter.h"
 
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "net/base/address_list.h"
 #include "net/base/net_errors.h"
@@ skipping 49 matching lines @@
     return net::ERR_UNEXPECTED;
   }
 
   // SSLConfigService is not thread-safe, and the default values for SSLConfig
   // are correct for us, so we don't use the config service to initialize this
   // object.
   net::SSLConfig ssl_config;
   transport_socket_->set_addr(talk_base::SocketAddress(hostname_, 0));
   ssl_socket_.reset(
       net::ClientSocketFactory::GetDefaultFactory()->CreateSSLClientSocket(
-          transport_socket_, hostname_.c_str(), ssl_config,
+          transport_socket_, hostname_.c_str(), /*TODO(rch)*/-1, ssl_config,

[wtc, 2010/11/11 01:11:35]

Isn't 443 better than -1 as a temporary port value?

[Ryan Hamilton, 2010/11/11 18:57:00]

Done.

           NULL /* ssl_host_info */));
 
   int result = ssl_socket_->Connect(&connected_callback_);
 
   if (result == net::ERR_IO_PENDING || result == net::OK) {
     return 0;
   } else {
     LOG(ERROR) << "Could not start SSL: " << net::ErrorToString(result);
     return result;
   }
@@ skipping 264 matching lines @@
         write_buffer_len_ = buffer_len;
         return;
       }
     }
     was_used_to_convey_data_ = true;
     callback->RunWithParams(Tuple1<int>(result));
   }
 }
 
 }  // namespace remoting