Remote RDP sessions, rather than the console, if curtain-mode is configured.

remoting/host/daemon_process.cc

 // Copyright (c) 2012 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/daemon_process.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
@@ skipping 156 matching lines @@
       next_terminal_id_(0),
       ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
   DCHECK(caller_task_runner->BelongsToCurrentThread());
 }
 
 void DaemonProcess::CreateDesktopSession(int terminal_id,
                                          const DesktopSessionParams& params,
                                          bool virtual_terminal) {
   DCHECK(caller_task_runner()->BelongsToCurrentThread());
 
+  // Terminal IDs cannot be reused. Update the expected next terminal ID.
+  next_terminal_id_ = std::max(next_terminal_id_, terminal_id + 1);
+
   // Validate the supplied terminal ID. An attempt to create a desktop session
   // with an ID that could possibly have been allocated already is considered
   // a protocol error and the network process will be restarted.
   if (IsTerminalIdKnown(terminal_id)) {
     LOG(ERROR) << "An invalid terminal ID. terminal_id=" << terminal_id;
     CrashNetworkProcess(FROM_HERE);
     return;
   }
 
   // Validate |params|.
   if (params.client_dpi_.x() < 0 || params.client_dpi_.y() < 0) {
     LOG(ERROR) << "Invalid DPI of the remote screen specified: "
                << params.client_dpi_;
     SendToNetwork(
         new ChromotingDaemonNetworkMsg_TerminalDisconnected(terminal_id));
     return;
   }
   if (params.client_size_.width() < 0 || params.client_size_.height() < 0) {
     LOG(ERROR) << "Invalid resolution of the remote screen specified: "
                << params.client_size_;
     SendToNetwork(
         new ChromotingDaemonNetworkMsg_TerminalDisconnected(terminal_id));
     return;
   }
 
   // Create the desktop session.
   scoped_ptr<DesktopSession> session = DoCreateDesktopSession(
       terminal_id, params, virtual_terminal);
-  if (session) {
-    VLOG(1) << "Daemon: opened desktop session " << terminal_id;
-    desktop_sessions_.push_back(session.release());
-  } else {
+  if (!session) {
     LOG(ERROR) << "Failed to create a desktop session.";
     SendToNetwork(
         new ChromotingDaemonNetworkMsg_TerminalDisconnected(terminal_id));
     return;
   }
 
-  // Update the expected terminal ID.
-  next_terminal_id_ = std::max(next_terminal_id_, terminal_id + 1);
+  VLOG(1) << "Daemon: opened desktop session " << terminal_id;
+  desktop_sessions_.push_back(session.release());
 }
 
 void DaemonProcess::CrashNetworkProcess(
     const tracked_objects::Location& location) {
   SendToNetwork(new ChromotingDaemonNetworkMsg_Crash(
       location.function_name(), location.file_name(), location.line_number()));
 
   DeleteAllDesktopSessions();
 }
 
@@ skipping 116 matching lines @@
 }
 
 void DaemonProcess::DeleteAllDesktopSessions() {
   while (!desktop_sessions_.empty()) {
     delete desktop_sessions_.front();
     desktop_sessions_.pop_front();
   }
 }
 
 }  // namespace remoting