commit 74036ae4a999fb36c6e08eaaebdb54ae1b8d3e50
author Ting-Yuan Huang <laszio@chromium.org> Fri Apr 08 17:03:34 2016 +0800
committer chrome-bot <chrome-bot@chromium.org> Mon Apr 11 04:15:04 2016 -0700
tree c5eae07719c379b089d1763d8c6add737277e71a
parent ac6fdbf28677b07290fe1951417354997b38d700

Fix parameters when generating commands to run via ssh

Use utils.sh_escape to prevent double quotes from being unfolded.

BUG=none
TEST=test_that --board=falco $DUT telemetry_Benchmarks.sunspider \
     --args="local=[True|False] telemetry_on_dut=[True|False]"

Change-Id: I4c344d3947fa0176c6c46bb9a582a06f0996f2ee
Reviewed-on: https://chromium-review.googlesource.com/337965
Commit-Ready: Ting-Yuan Huang <laszio@chromium.org>
Tested-by: Ting-Yuan Huang <laszio@chromium.org>
Reviewed-by: Dan Shi <dshi@google.com>

server/cros/telemetry_runner.py

diff --git a/server/cros/telemetry_runner.py b/server/cros/telemetry_runner.py
index 8c43721..0b695e5 100644
--- a/server/cros/telemetry_runner.py
+++ b/server/cros/telemetry_runner.py
@@ -29,6 +29,8 @@
 DUT_RSYNC_OPTIONS =  ' '.join(['--rsh="/usr/bin/ssh %s"' % DUT_SSH_OPTIONS,
                                '-L', '--timeout=1800', '-az',
                                '--no-o', '--no-g'])
+# Prevent double quotes from being unfolded.
+DUT_RSYNC_OPTIONS = utils.sh_escape(DUT_RSYNC_OPTIONS)
 
 # Result Statuses
 SUCCESS_STATUS = 'SUCCESS'