Fix GnubbyAuthHandlerPosix to handle requests correctly.

remoting/host/gnubby_auth_handler_posix.cc

 // Copyright 2014 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/host/gnubby_auth_handler_posix.h"
 
 #include <unistd.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 213 matching lines @@
   DCHECK(CalledOnValidThread());
 
   ActiveSockets::iterator iter = active_sockets_.find(connection_id);
   DCHECK(iter != active_sockets_.end());
   std::string request_data;
   if (!iter->second->GetAndClearRequestData(&request_data)) {
     SendErrorAndCloseActiveSocket(iter);
     return;
   }
   ProcessGnubbyRequest(connection_id, request_data);
-  Close(iter);
+  iter->second->StartReadingRequest(
+      base::Bind(&GnubbyAuthHandlerPosix::OnReadComplete,
+                 base::Unretained(this), connection_id));
 }
 
 void GnubbyAuthHandlerPosix::Close(const ActiveSockets::iterator& iter) {
   DCHECK(CalledOnValidThread());
 
   delete iter->second;
   active_sockets_.erase(iter);

[Sergey Ulanov, 2015/07/16 23:20:54]

I think this can be moved in SendErrorAndCloseActiveSocket() and Close() can be
removed.

[Wez, 2015/07/17 20:51:19]

Done.

 }
 
 void GnubbyAuthHandlerPosix::CreateAuthorizationSocket() {
   DCHECK(CalledOnValidThread());
 
   if (!g_gnubby_socket_name.Get().empty()) {
     // TODO(wez): Should creation of the Unix-domain socket be punted to an
     // I/O thread? See crbug.com/509807.
     {
       base::ThreadRestrictions::ScopedAllowIO allow_io;
@@ skipping 42 matching lines @@
 }
 
 void GnubbyAuthHandlerPosix::RequestTimedOut(int connection_id) {
   HOST_LOG << "Gnubby request timed out";
   ActiveSockets::iterator iter = active_sockets_.find(connection_id);
   if (iter != active_sockets_.end())
     SendErrorAndCloseActiveSocket(iter);
 }
 
 }  // namespace remoting