From 5cf0fec72a613b5a64971c4e862b4d239471bf4b Mon Sep 17 00:00:00 2001 From: Tim Newsome Date: Wed, 7 Dec 2016 17:52:26 -0800 Subject: [PATCH] Fix race when finding the port OpenOCD is using. --- debug/testlib.py | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/debug/testlib.py b/debug/testlib.py index b8e9ad4..6b01d8d 100644 --- a/debug/testlib.py +++ b/debug/testlib.py @@ -165,14 +165,19 @@ class Openocd(object): PORT_REGEX = re.compile(r'(?P\d+) \(LISTEN\)') for _ in range(MAX_ATTEMPTS): with open(os.devnull, 'w') as devnull: - output = subprocess.check_output([ - 'lsof', - '-a', # Take the AND of the following selectors - '-p{}'.format(self.process.pid), # Filter on PID - '-iTCP', # Filter only TCP sockets - ], stderr=devnull) + try: + output = subprocess.check_output([ + 'lsof', + '-a', # Take the AND of the following selectors + '-p{}'.format(self.process.pid), # Filter on PID + '-iTCP', # Filter only TCP sockets + ], stderr=devnull) + except subprocess.CalledProcessError: + output = "" matches = list(PORT_REGEX.finditer(output)) + matches = [m for m in matches if m.group('port') not in ('6666', '4444')] if len(matches) > 1: + print output raise Exception( "OpenOCD listening on multiple ports. Cannot uniquely " "identify gdb server port.") -- 2.30.2