Reverting 19466.

net/socket/socks_client_socket.cc

 // Copyright (c) 2009 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 "net/socket/socks_client_socket.h"
 
 #include "base/basictypes.h"
 #include "build/build_config.h"
 #if defined(OS_WIN)
 #include <ws2tcpip.h>
@@ skipping 10 matching lines @@
 // Every SOCKS server requests a user-id from the client. It is optional
 // and we send an empty string.
 static const char kEmptyUserId[] = "";
 
 // The SOCKS4a implementation suggests to use an invalid IP in case the DNS
 // resolution at client fails.
 static const uint8 kInvalidIp[] = { 0, 0, 0, 127 };
 
 // For SOCKS4, the client sends 8 bytes  plus the size of the user-id.
 // For SOCKS4A, this increases to accomodate the unresolved hostname.
-static const unsigned int kWriteHeaderSize = 8;
+static const int kWriteHeaderSize = 8;
 
 // For SOCKS4 and SOCKS4a, the server sends 8 bytes for acknowledgement.
-static const unsigned int kReadHeaderSize = 8;
+static const int kReadHeaderSize = 8;
 
 // Server Response codes for SOCKS.
 static const uint8 kServerResponseOk  = 0x5A;
 static const uint8 kServerResponseRejected = 0x5B;
 static const uint8 kServerResponseNotReachable = 0x5C;
 static const uint8 kServerResponseMismatchedUserId = 0x5D;
 
 static const uint8 kSOCKSVersion4 = 0x04;
 static const uint8 kSOCKSStreamRequest = 0x01;
 
@@ skipping 20 matching lines @@
 
 SOCKSClientSocket::SOCKSClientSocket(ClientSocket* transport_socket,
                                      const HostResolver::RequestInfo& req_info,
                                      HostResolver* host_resolver)
     : ALLOW_THIS_IN_INITIALIZER_LIST(
           io_callback_(this, &SOCKSClientSocket::OnIOComplete)),
       transport_(transport_socket),
       next_state_(STATE_NONE),
       socks_version_(kSOCKS4Unresolved),
       user_callback_(NULL),
+      handshake_buf_len_(0),
+      buffer_(NULL),
+      buffer_len_(0),
       completed_handshake_(false),
       bytes_sent_(0),
       bytes_received_(0),
       resolver_(host_resolver),
       host_request_info_(req_info) {
 }
 
 SOCKSClientSocket::~SOCKSClientSocket() {
   Disconnect();
 }
@@ skipping 138 matching lines @@
   }
 
   // Even if DNS resolution fails, we send OK since the server
   // resolves the domain.
   return OK;
 }
 
 // Builds the buffer that is to be sent to the server.
 // We check whether the SOCKS proxy is 4 or 4A.
 // In case it is 4A, the record size increases by size of the hostname.
-const std::string SOCKSClientSocket::BuildHandshakeWriteBuffer() const {
+void SOCKSClientSocket::BuildHandshakeWriteBuffer() {
   DCHECK_NE(kSOCKS4Unresolved, socks_version_);
 
-  SOCKS4ServerRequest request;
-  request.version = kSOCKSVersion4;
-  request.command = kSOCKSStreamRequest;
-  request.nw_port = htons(host_request_info_.port());
+  int record_size = kWriteHeaderSize + arraysize(kEmptyUserId);
+  if (socks_version_ == kSOCKS4a) {
+    record_size += host_request_info_.hostname().size() + 1;
+  }
+
+  buffer_len_ = record_size;
+  buffer_.reset(new char[buffer_len_]);
+
+  SOCKS4ServerRequest* request =
+      reinterpret_cast<SOCKS4ServerRequest*>(buffer_.get());
+
+  request->version = kSOCKSVersion4;
+  request->command = kSOCKSStreamRequest;
+  request->nw_port = htons(host_request_info_.port());
 
   if (socks_version_ == kSOCKS4) {
     const struct addrinfo* ai = addresses_.head();
     DCHECK(ai);
     // If the sockaddr is IPv6, we have already marked the version to socks4a
     // and so this step does not get hit.
-    struct sockaddr_in* ipv4_host =
+    struct sockaddr_in *ipv4_host =
         reinterpret_cast<struct sockaddr_in*>(ai->ai_addr);
-    memcpy(&request.ip, &(ipv4_host->sin_addr), sizeof(ipv4_host->sin_addr));
+    memcpy(&request->ip, &(ipv4_host->sin_addr), sizeof(ipv4_host->sin_addr));
 
     DLOG(INFO) << "Resolved Host is : " << NetAddressToString(ai);
   } else if (socks_version_ == kSOCKS4a) {
     // invalid IP of the form 0.0.0.127
-    memcpy(&request.ip, kInvalidIp, arraysize(kInvalidIp));
+    memcpy(&request->ip, kInvalidIp, arraysize(kInvalidIp));
   } else {
     NOTREACHED();
   }
 
-  std::string handshake_data(reinterpret_cast<char*>(&request),
-                             sizeof(request));
-  handshake_data.append(kEmptyUserId, arraysize(kEmptyUserId));
+  memcpy(&buffer_[kWriteHeaderSize], kEmptyUserId, arraysize(kEmptyUserId));
 
-  // In case we are passing the domain also, pass the hostname
-  // terminated with a null character.
   if (socks_version_ == kSOCKS4a) {
-    handshake_data.append(host_request_info_.hostname());
-    handshake_data.push_back('\0');
+    memcpy(&buffer_[kWriteHeaderSize + arraysize(kEmptyUserId)],
+           host_request_info_.hostname().c_str(),
+           host_request_info_.hostname().size() + 1);
   }
-
-  return handshake_data;
 }
 
 // Writes the SOCKS handshake data to the underlying socket connection.
 int SOCKSClientSocket::DoHandshakeWrite() {
   next_state_ = STATE_HANDSHAKE_WRITE_COMPLETE;
 
-  if (buffer_.empty()) {
-    buffer_ = BuildHandshakeWriteBuffer();
+  if (!buffer_.get()) {
+    BuildHandshakeWriteBuffer();
     bytes_sent_ = 0;
   }
 
-  int handshake_buf_len = buffer_.size() - bytes_sent_;
-  DCHECK_GT(handshake_buf_len, 0);
-  handshake_buf_ = new IOBuffer(handshake_buf_len);
-  memcpy(handshake_buf_->data(), &buffer_[bytes_sent_],
-         handshake_buf_len);
-  return transport_->Write(handshake_buf_, handshake_buf_len, &io_callback_);
+  handshake_buf_len_ = buffer_len_ - bytes_sent_;
+  DCHECK_GT(handshake_buf_len_, 0);
+  handshake_buf_ = new IOBuffer(handshake_buf_len_);
+  memcpy(handshake_buf_.get()->data(), &buffer_[bytes_sent_],
+         handshake_buf_len_);
+  return transport_->Write(handshake_buf_, handshake_buf_len_, &io_callback_);
 }
 
 int SOCKSClientSocket::DoHandshakeWriteComplete(int result) {
   DCHECK_NE(kSOCKS4Unresolved, socks_version_);
 
   if (result < 0)
     return result;
 
-  // We ignore the case when result is 0, since the underlying Write
-  // may return spurious writes while waiting on the socket.
-
   bytes_sent_ += result;
-  if (bytes_sent_ == buffer_.size()) {
+  if (bytes_sent_ == buffer_len_) {
     next_state_ = STATE_HANDSHAKE_READ;
-    buffer_.clear();
-  } else if (bytes_sent_ < buffer_.size()) {
+    buffer_.reset(NULL);
+  } else if (bytes_sent_ < buffer_len_) {
     next_state_ = STATE_HANDSHAKE_WRITE;
   } else {
     return ERR_UNEXPECTED;
   }
 
   return OK;
 }
 
 int SOCKSClientSocket::DoHandshakeRead() {
   DCHECK_NE(kSOCKS4Unresolved, socks_version_);
 
   next_state_ = STATE_HANDSHAKE_READ_COMPLETE;
 
-  if (buffer_.empty()) {
+  if (!buffer_.get()) {
+    buffer_.reset(new char[kReadHeaderSize]);
+    buffer_len_ = kReadHeaderSize;
     bytes_received_ = 0;
   }
 
-  int handshake_buf_len = kReadHeaderSize - bytes_received_;
-  handshake_buf_ = new IOBuffer(handshake_buf_len);
-  return transport_->Read(handshake_buf_, handshake_buf_len, &io_callback_);
+  handshake_buf_len_ = buffer_len_ - bytes_received_;
+  handshake_buf_ = new IOBuffer(handshake_buf_len_);
+  return transport_->Read(handshake_buf_, handshake_buf_len_, &io_callback_);
 }
 
 int SOCKSClientSocket::DoHandshakeReadComplete(int result) {
   DCHECK_NE(kSOCKS4Unresolved, socks_version_);
 
   if (result < 0)
     return result;
-
-  // The underlying socket closed unexpectedly.
-  if (result == 0)
-    return ERR_CONNECTION_CLOSED;
-
-  if (bytes_received_ + result > kReadHeaderSize)
+  if (bytes_received_ + result > buffer_len_)
     return ERR_INVALID_RESPONSE;
 
-  buffer_.append(handshake_buf_->data(), result);
+  memcpy(buffer_.get() + bytes_received_, handshake_buf_->data(), result);
   bytes_received_ += result;
-  if (bytes_received_ < kReadHeaderSize) {
+  if (bytes_received_ < buffer_len_) {
     next_state_ = STATE_HANDSHAKE_READ;
     return OK;
   }
 
-  const SOCKS4ServerResponse* response =
-      reinterpret_cast<const SOCKS4ServerResponse*>(buffer_.data());
+  SOCKS4ServerResponse* response =
+      reinterpret_cast<SOCKS4ServerResponse*>(buffer_.get());
 
   if (response->reserved_null != 0x00) {
     LOG(ERROR) << "Unknown response from SOCKS server.";
     return ERR_INVALID_RESPONSE;
   }
 
   // TODO(arindam): Add SOCKS specific failure codes in net_error_list.h
   switch (response->code) {
     case kServerResponseOk:
       completed_handshake_ = true;
@@ skipping 22 matching lines @@
 // Needed by ssl_client_socket_nss.
 int SOCKSClientSocket::GetPeerName(struct sockaddr *name, socklen_t *namelen) {
   // Default implementation just permits some unit tests to link.
   NOTREACHED();
   return ERR_UNEXPECTED;
 }
 #endif
 
 }  // namespace net