commit 8cbd3f431a919e3398683bb5b72fdd781798bcc8
author Artem Dergachev <artem.dergachev@gmail.com> Wed May 13 14:06:44 2020 +0300
committer Artem Dergachev <artem.dergachev@gmail.com> Wed May 13 14:13:00 2020 +0300
tree 3710179992de9834eb36cf7b132c455139b9e9f2
parent e16111ce2fce7fd86c10d3f1dfe3e3b62b76d73b

[analyzer] SATestBuild.py: Be defensive against corrupt plist files.

clang/utils/analyzer/SATestBuild.py

diff --git a/clang/utils/analyzer/SATestBuild.py b/clang/utils/analyzer/SATestBuild.py
index 347f1e2..42b2658 100755
--- a/clang/utils/analyzer/SATestBuild.py
+++ b/clang/utils/analyzer/SATestBuild.py
@@ -58,6 +58,7 @@
 import sys
 import threading
 import time
+from xml.parsers.expat import ExpatError
 try:
     import queue
 except ImportError:
@@ -485,10 +486,14 @@
     for F in glob.glob(SBOutputDir + "/*/*.plist"):
         P = os.path.join(SBOutputDir, F)
 
-        Data = plistlib.readPlist(P)
-        # Delete empty reports.
-        if not Data['files']:
-            os.remove(P)
+        try:
+            Data = plistlib.readPlist(P)
+            # Delete empty reports.
+            if not Data['files']:
+                os.remove(P)
+                continue
+        except ExpatError as e:
+            print('Error parsing plist file %s: %s' % (P, str(e)))
             continue