Skip to content

Commit 1a25dbb

Browse files
DeviceInfracopybara-github
authored andcommitted
Internal change
PiperOrigin-RevId: 875500873
1 parent ce8f79f commit 1a25dbb

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/java/com/google/devtools/mobileharness/platform/android/instrumentation/AndroidInstrumentationUtil.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@
5555
import com.google.devtools.mobileharness.shared.util.file.local.LocalFileUtil;
5656
import com.google.devtools.mobileharness.shared.util.file.local.ResUtil;
5757
import com.google.devtools.mobileharness.shared.util.path.PathUtil;
58+
import com.google.devtools.mobileharness.shared.util.shell.ShellUtils;
5859
import com.google.devtools.mobileharness.shared.util.time.Sleeper;
5960
import com.google.errorprone.annotations.CanIgnoreReturnValue;
6061
import com.google.wireless.qa.mobileharness.shared.android.Aapt;
@@ -389,15 +390,15 @@ public String instrument(
389390
}
390391
if (!StrUtil.isEmptyOrWhitespace(className)) {
391392
command.append(" -e class ");
392-
command.append(className);
393+
command.append(ShellUtils.shellEscape(className));
393394
}
394395

395396
if (otherOptions != null && !otherOptions.isEmpty()) {
396397
for (Entry<String, String> entry : otherOptions.entrySet()) {
397398
command.append(" -e ");
398-
command.append(entry.getKey());
399+
command.append(ShellUtils.shellEscape(entry.getKey()));
399400
command.append(' ');
400-
command.append(entry.getValue());
401+
command.append(ShellUtils.shellEscape(entry.getValue()));
401402
}
402403
}
403404

src/java/com/google/devtools/mobileharness/platform/android/instrumentation/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ java_library(
5656
"//src/java/com/google/devtools/mobileharness/shared/util/file/local:res_util",
5757
"//src/java/com/google/devtools/mobileharness/shared/util/logging:google_logger",
5858
"//src/java/com/google/devtools/mobileharness/shared/util/path",
59+
"//src/java/com/google/devtools/mobileharness/shared/util/shell",
5960
"//src/java/com/google/devtools/mobileharness/shared/util/time:sleeper",
6061
"//src/java/com/google/wireless/qa/mobileharness/shared/android:aapt",
6162
"//src/java/com/google/wireless/qa/mobileharness/shared/api/device",

0 commit comments

Comments
 (0)