michaelh/zephyr
1
0
Fork 0
Code Releases Activity

sanitycheck: close pipe file handles

Browse source 
Make sure we close the pipe file handles after we are done, otherwise we
will end up with too many open file descriptors and crash...

Fixes #20974
Fixes #21637

Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This commit is contained in:
Anas Nashif 2020-01-03 16:29:46 -05:00
commit bdffb88950
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
scripts/sanitycheck
View file

@ -812,12 +812,15 @@ class DeviceHandler(Handler):
t.join(self.timeout) t.join(self.timeout)
if t.is_alive(): if t.is_alive():
logger.debug("Timed out") logger.debug("Timed out while monitoring serial output on {}".format(self.instance.platform.name))
out_state = "timeout" out_state = "timeout"
if ser.isOpen(): if ser.isOpen():
ser.close() ser.close()
os.close(write_pipe)
os.close(read_pipe)
handler_time = time.time() - start_time handler_time = time.time() - start_time
if out_state == "timeout": if out_state == "timeout":