[U-Boot] [PATCH 1/1] test/py: catch errors occuring when reading the console

Stephen Warren swarren at wwwdotorg.org
Tue Sep 18 17:06:43 UTC 2018


On 09/17/2018 04:30 PM, Heinrich Schuchardt wrote:
> Spawn.exept has a try block without 'except'.
> 
> If no output is available an OSError may arise. Catch this exception and
> continue testing.

> diff --git a/test/py/u_boot_spawn.py b/test/py/u_boot_spawn.py
> index b011a3e3da..3c18360e62 100644
> --- a/test/py/u_boot_spawn.py
> +++ b/test/py/u_boot_spawn.py
> @@ -181,6 +181,9 @@ class Spawn(object):
>                   # unlimited substitutions, but in practice the version of
>                   # Python in Ubuntu 14.04 appears to default to count=2!
>                   self.buf = self.re_vt100.sub('', self.buf, count=1000000)
> +        except OSError, EOFError:
> +	    # Reading the the console may result in an error. Catch it.

This line is indented wrong; it mixes in a TAB instead of using spaces.

> +            pass
>           finally:
>               if self.logfile_read:
>                   self.logfile_read.flush()
> 

This doesn't make sense at all. It catches all errors and ignores them. 
It'll turn any error condition into a timeout (presumably, the expected 
data being waited for will never appear) rather than dealing with it 
immediately (due to the thrown exception). Why is this needed?


More information about the U-Boot mailing list