Handle Security Key requests from outside the remoted session correctly

remoting/host/security_key/security_key_ipc_server_impl.cc

 // Copyright 2016 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/security_key/security_key_ipc_server_impl.h"
 
 #include <cstdint>
 #include <memory>
 #include <string>
 
@@ skipping 143 matching lines @@
 #if defined(OS_WIN)
   DWORD peer_session_id;
   if (!ProcessIdToSessionId(peer_pid, &peer_session_id)) {
     PLOG(ERROR) << "ProcessIdToSessionId() failed";
     connection_close_pending_ = true;
   } else if (peer_session_id != client_session_details_->desktop_session_id()) {
     LOG(ERROR) << "Ignoring connection attempt from outside remoted session.";
     connection_close_pending_ = true;
   }
   if (connection_close_pending_) {
+    ipc_channel_->Send(
+        new ChromotingNetworkToRemoteSecurityKeyMsg_InvalidSession());
+
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE, base::Bind(&SecurityKeyIpcServerImpl::OnChannelError,
                               weak_factory_.GetWeakPtr()));
     return;
   }
 #else   // !defined(OS_WIN)
   CHECK_EQ(client_session_details_->desktop_session_id(), UINT32_MAX);
 #endif  // !defined(OS_WIN)
 
   // Reset the timer to give the client a chance to send the request.
   timer_.Start(FROM_HERE, initial_connect_timeout_,
                base::Bind(&SecurityKeyIpcServerImpl::OnChannelError,
                           base::Unretained(this)));
+
+  ipc_channel_->Send(
+      new ChromotingNetworkToRemoteSecurityKeyMsg_ConnectionReady());
 }
 
 void SecurityKeyIpcServerImpl::OnChannelError() {
   DCHECK(thread_checker_.CalledOnValidThread());
   CloseChannel();
 
   if (!connect_callback_.is_null()) {
     base::ResetAndReturn(&connect_callback_).Run();
   }
   if (!done_callback_.is_null()) {
@@ skipping 21 matching lines @@
     connection_close_pending_ = false;
   }
   // Close the underlying mojo connection.
   if (!mojo_peer_token_.empty()) {
     mojo::edk::ClosePeerConnection(mojo_peer_token_);
     mojo_peer_token_.clear();
   }
 }
 
 }  // namespace remoting