@@ -278,6 +278,7 @@ def handle_return(self, default_timeout):
278278 self .out , self .err = self .process .communicate (timeout = default_timeout )
279279 except subprocess .TimeoutExpired :
280280 INFO (1 , None , None , "Timeout occured" )
281+ kill_all (self .process .pid )
281282 return False
282283 else :
283284 self .out , self .err = self .process .communicate () # watchdog will handle timeout if needed in PY2
@@ -306,16 +307,6 @@ def exec_command(self, cmd):
306307 INFO (1 , None , None , "Target successfully started, waiting for result" )
307308 self .handle_return (self .timeout )
308309
309-
310- if PY3 :
311- try :
312- self .out , self .err = self .process .communicate (timeout = self .timeout )
313- except subprocess .TimeoutExpired :
314- INFO (1 , None , None , "Timeout for %s" % cmd )
315- kill_all (self .process .pid )
316- else :
317- self .out , self .err = self .process .communicate () # watchdog will handle timeout if needed
318-
319310 def run (self , cmd ):
320311
321312 self .exec_command (cmd )
@@ -1323,7 +1314,7 @@ def parse_args():
13231314 parser .add_argument ('-r' , default = False , action = 'store_true' , dest = "restore" , help = "Restore previous session" )
13241315
13251316 # these options should be specified through config file and hidden
1326- parser .add_argument ('--determinstic_seed ' , default = False , action = 'store_true' , help = argparse .SUPPRESS )
1317+ parser .add_argument ('--deterministic_seed ' , default = False , action = 'store_true' , help = argparse .SUPPRESS )
13271318 parser .add_argument ('--print_per_thread' , default = False , action = 'store_true' , dest = "threads_info" , help = argparse .SUPPRESS )
13281319
13291320 parser .add_argument ('--dbi' , default = None , help = argparse .SUPPRESS )
0 commit comments