diff mbox series

[kirkstone,19/21] scripts/lib/buildstats: handle top-level build_stats not being complete

Message ID 19815bb76e414a1c1fd8d31a84492d72dd29205e.1680453201.git.steve@sakoman.com
State New, archived
Headers show
Series [kirkstone,01/21] Revert "qemu: fix CVE-2021-3507" | expand

Commit Message

Steve Sakoman April 2, 2023, 4:42 p.m. UTC 
From: Ross Burton <ross.burton@arm.com>

If we try to parse a buildstats directory which was either aborted or
is still being built then the top-level build_stats file doesn't
contain an elapsed value which causes an exception:

UnboundLocalError: local variable 'elapsed' referenced before assignment

Default both start and elapsed to 0 so that the parse succeeds.

Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
(cherry picked from commit 701d985aa8f2e9c2b9c0736fa25b424f3701889e)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 scripts/lib/buildstats.py | 1 +
 1 file changed, 1 insertion(+)
diff mbox series

Patch

diff --git a/scripts/lib/buildstats.py b/scripts/lib/buildstats.py
index fa94c65539..6db60d5bcf 100644
--- a/scripts/lib/buildstats.py
+++ b/scripts/lib/buildstats.py
@@ -234,6 +234,7 @@  class BuildStats(dict):
         """
         Parse the top-level build_stats file for build-wide start and duration.
         """
+        start = elapsed = 0
         with open(path) as fobj:
             for line in fobj.readlines():
                 key, val = line.split(':', 1)