[Android] Run shell commands from the forwarder without passing fds.

tools/android/forwarder2/host_forwarder_main.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 <errno.h>
 #include <signal.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <sys/types.h>
 #include <sys/wait.h>
@@ skipping 11 matching lines @@
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/containers/hash_tables.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/pickle.h"
+#include "base/process/launch.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/task_runner.h"
 #include "base/threading/thread.h"
 #include "tools/android/forwarder2/common.h"
 #include "tools/android/forwarder2/daemon.h"
 #include "tools/android/forwarder2/host_controller.h"
@@ skipping 269 matching lines @@
     // - Remove from "adb forward" command.
     LOG(INFO) << "Device " << device_serial << " has no more ports.";
     device_serial_to_adb_port_map_.erase(device_serial);
     const std::string serial_part = device_serial.empty() ?
         std::string() : std::string("-s ") + device_serial;
     const std::string command = base::StringPrintf(
         "%s %s forward --remove tcp:%d",
         adb_path.c_str(),
         serial_part.c_str(),
         port);
-    const int ret = system(command.c_str());
-    LOG(INFO) << command << " ret: " << ret;
+    const base::CommandLine command_line(base::SplitString(
+        command, " ", base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY));
+    int ret;
+    std::string output;
+    base::GetAppOutputWithExitCode(command_line, &output, &ret);
+    LOG(INFO) << command << " ret: " << ret << " output: " << output;
     // Wait for the socket to be fully unmapped.
     const std::string port_mapped_cmd = base::StringPrintf(
         "lsof -nPi:%d",
         port);
+    const base::CommandLine port_mapped_cmd_line(
+        base::SplitString(port_mapped_cmd, " ", base::TRIM_WHITESPACE,
+                          base::SPLIT_WANT_NONEMPTY));
     const int poll_interval_us = 500 * 1000;
     int retries = 3;
     while (retries) {
-      const int port_unmapped = system(port_mapped_cmd.c_str());
+      int port_unmapped;
+      base::GetAppOutputWithExitCode(port_mapped_cmd_line, &output,
+                                     &port_unmapped);
       LOG(INFO) << "Device " << device_serial << " port " << port << " unmap "
                 << port_unmapped;
       if (port_unmapped)
         break;
       --retries;
       usleep(poll_interval_us);
     }
   }
 
   int GetAdbPortForDevice(const std::string adb_path,
                           const std::string& device_serial) {
     base::hash_map<std::string, int>::const_iterator it =
         device_serial_to_adb_port_map_.find(device_serial);
     if (it != device_serial_to_adb_port_map_.end())
       return it->second;
     Socket bind_socket;
     CHECK(bind_socket.BindTcp("127.0.0.1", 0));
     const int port = bind_socket.GetPort();
     bind_socket.Close();
     const std::string serial_part = device_serial.empty() ?
         std::string() : std::string("-s ") + device_serial;
     const std::string command = base::StringPrintf(
         "%s %s forward tcp:%d localabstract:chrome_device_forwarder",
         adb_path.c_str(),
         serial_part.c_str(),
         port);
-    LOG(INFO) << command;
-    const int ret = system(command.c_str());
+    const base::CommandLine command_line(base::SplitString(
+        command, " ", base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY));
+    int ret;
+    std::string output;
+    base::GetAppOutputWithExitCode(command_line, &output, &ret);
+    LOG(INFO) << command << " ret: " << ret << " output: " << output;
     if (ret < 0 || !WIFEXITED(ret) || WEXITSTATUS(ret) != 0)
       return -1;
     device_serial_to_adb_port_map_[device_serial] = port;
     return port;
   }
 
   bool SendMessage(const std::string& msg, Socket* client_socket) {
     bool result = client_socket->WriteString(msg);
     DCHECK(result);
     if (!result)
@@ skipping 177 matching lines @@
 
   return client_delegate.has_failed() || daemon_delegate.has_failed();
 }
 
 }  // namespace
 }  // namespace forwarder2
 
 int main(int argc, char** argv) {
   return forwarder2::RunHostForwarder(argc, argv);
 }