Uses a lock around Popens() to prevent a deadlock in Python.
Please see crbug.com/323843 for more details.
BUG=chromium:323843
TEST=Run the audiovideo_CRASFormatConversion on parrot
Change-Id: I12b9450c937acf74888ed995f25f405c4f15a705
Reviewed-on: https://chromium-review.googlesource.com/178373
Reviewed-by: Owen Lin <owenlin@chromium.org>
Tested-by: Owen Lin <owenlin@chromium.org>
Commit-Queue: Owen Lin <owenlin@chromium.org>
diff --git a/client/cros/audio/cras_utils.py b/client/cros/audio/cras_utils.py
index d99c157..74422de 100644
--- a/client/cros/audio/cras_utils.py
+++ b/client/cros/audio/cras_utils.py
@@ -4,6 +4,7 @@
import logging
import re
+import subprocess
from autotest_lib.client.bin import utils
from autotest_lib.client.cros.audio import cmd_utils
@@ -83,7 +84,8 @@
def dump_server_info():
"""Gets the CRAS's server information."""
- return utils.system_output('%s --dump_server_info' % _CRAS_TEST_CLIENT)
+ args = [_CRAS_TEST_CLIENT, '--dump_server_info']
+ return cmd_utils.execute(args, stdout=subprocess.PIPE)
def get_selected_nodes():
"""Returns the pair of active output node and input node."""