From 68d1a6301416e7056d2c8f39e9a05eb3556d44f3 Mon Sep 17 00:00:00 2001
From: Anton Romanov <romanton@google.com>
Date: Tue, 1 Mar 2022 22:09:54 +0000
Subject: [PATCH] Prevent dev_container arguments from being split

BUG=none
TEST=./tools/dev_container --hermetic bash -c "echo true &&./tools/clippy"

Change-Id: If0bd7b13ae4825e0bdaba9db06e6ec9a80ec3541
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3498847
Reviewed-by: Anton Romanov <romanton@google.com>
Commit-Queue: Anton Romanov <romanton@google.com>
Auto-Submit: Anton Romanov <romanton@google.com>
Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
Reviewed-by: Dennis Kempin <denniskempin@google.com>
Commit-Queue: Dennis Kempin <denniskempin@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
---
 tools/dev_container | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tools/dev_container b/tools/dev_container
index 1c4e0c9a11..0128508d15 100755
--- a/tools/dev_container
+++ b/tools/dev_container
@@ -75,6 +75,8 @@ def main(command: tuple[str, ...], stop: bool = False, hermetic: bool = False):
     if not command:
         command = ("/bin/bash",)
 
+    command = list(map(quoted, command))
+
     if stop:
         if container_id:
             print(f"Stopping dev-container {container_id}.")