Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 1 | #!/usr/bin/env python |
| 2 | |
| 3 | """ |
| 4 | Static Analyzer qualification infrastructure. |
| 5 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 6 | The goal is to test the analyzer against different projects, |
| 7 | check for failures, compare results, and measure performance. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 8 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 9 | Repository Directory will contain sources of the projects as well as the |
| 10 | information on how to build them and the expected output. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 11 | Repository Directory structure: |
| 12 | - ProjectMap file |
| 13 | - Historical Performance Data |
| 14 | - Project Dir1 |
| 15 | - ReferenceOutput |
| 16 | - Project Dir2 |
| 17 | - ReferenceOutput |
| 18 | .. |
Gabor Horvath | c3177f2 | 2015-07-08 18:39:31 +0000 | [diff] [blame] | 19 | Note that the build tree must be inside the project dir. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 20 | |
| 21 | To test the build of the analyzer one would: |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 22 | - Copy over a copy of the Repository Directory. (TODO: Prefer to ensure that |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 23 | the build directory does not pollute the repository to min network |
| 24 | traffic). |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 25 | - Build all projects, until error. Produce logs to report errors. |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 26 | - Compare results. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 27 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 28 | The files which should be kept around for failure investigations: |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 29 | RepositoryCopy/Project DirI/ScanBuildResults |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 30 | RepositoryCopy/Project DirI/run_static_analyzer.log |
| 31 | |
| 32 | Assumptions (TODO: shouldn't need to assume these.): |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 33 | The script is being run from the Repository Directory. |
Anna Zaks | 42a4463 | 2011-11-02 20:46:50 +0000 | [diff] [blame] | 34 | The compiler for scan-build and scan-build are in the PATH. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 35 | export PATH=/Users/zaks/workspace/c2llvm/build/Release+Asserts/bin:$PATH |
| 36 | |
| 37 | For more logging, set the env variables: |
| 38 | zaks:TI zaks$ export CCC_ANALYZER_LOG=1 |
| 39 | zaks:TI zaks$ export CCC_ANALYZER_VERBOSE=1 |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 40 | |
Gabor Horvath | da32a86 | 2015-08-20 22:59:49 +0000 | [diff] [blame] | 41 | The list of checkers tested are hardcoded in the Checkers variable. |
| 42 | For testing additional checkers, use the SA_ADDITIONAL_CHECKERS environment |
| 43 | variable. It should contain a comma separated list. |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 44 | """ |
| 45 | import CmpRuns |
George Karpenkov | bf92c44 | 2017-10-24 23:52:48 +0000 | [diff] [blame] | 46 | import SATestUtils |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 47 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 48 | from subprocess import CalledProcessError, check_call |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 49 | import argparse |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 50 | import csv |
| 51 | import glob |
| 52 | import logging |
| 53 | import math |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 54 | import multiprocessing |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 55 | import os |
| 56 | import plistlib |
| 57 | import shutil |
| 58 | import sys |
| 59 | import threading |
| 60 | import time |
Artem Dergachev | 1a3b801 | 2020-05-15 14:27:30 +0300 | [diff] [blame^] | 61 | |
Serge Guelton | 1f88dc5 | 2018-12-13 07:44:19 +0000 | [diff] [blame] | 62 | try: |
| 63 | import queue |
| 64 | except ImportError: |
| 65 | import Queue as queue |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 66 | |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 67 | ############################################################################### |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 68 | # Helper functions. |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 69 | ############################################################################### |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 70 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 71 | Local = threading.local() |
| 72 | Local.stdout = sys.stdout |
| 73 | Local.stderr = sys.stderr |
| 74 | logging.basicConfig( |
| 75 | level=logging.DEBUG, |
| 76 | format='%(asctime)s:%(levelname)s:%(name)s: %(message)s') |
Ted Kremenek | f9a539d | 2012-08-28 20:40:04 +0000 | [diff] [blame] | 77 | |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 78 | |
| 79 | class StreamToLogger: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 80 | def __init__(self, logger, log_level=logging.INFO): |
| 81 | self.logger = logger |
| 82 | self.log_level = log_level |
| 83 | |
| 84 | def write(self, buf): |
| 85 | # Rstrip in order not to write an extra newline. |
| 86 | self.logger.log(self.log_level, buf.rstrip()) |
| 87 | |
| 88 | def flush(self): |
| 89 | pass |
| 90 | |
| 91 | def fileno(self): |
| 92 | return 0 |
Anna Zaks | de1f7f8b | 2012-01-10 18:10:25 +0000 | [diff] [blame] | 93 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 94 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 95 | def getProjectMapPath(): |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 96 | ProjectMapPath = os.path.join(os.path.abspath(os.curdir), |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 97 | ProjectMapFile) |
| 98 | if not os.path.exists(ProjectMapPath): |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 99 | Local.stdout.write("Error: Cannot find the Project Map file " + |
| 100 | ProjectMapPath + |
| 101 | "\nRunning script for the wrong directory?\n") |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 102 | sys.exit(1) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 103 | return ProjectMapPath |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 104 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 105 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 106 | def getProjectDir(ID): |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 107 | return os.path.join(os.path.abspath(os.curdir), ID) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 108 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 109 | |
| 110 | def getSBOutputDirName(IsReferenceBuild): |
| 111 | if IsReferenceBuild: |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 112 | return SBOutputDirReferencePrefix + SBOutputDirName |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 113 | else: |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 114 | return SBOutputDirName |
| 115 | |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 116 | ############################################################################### |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 117 | # Configuration setup. |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 118 | ############################################################################### |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 119 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 120 | |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 121 | # Find Clang for static analysis. |
George Karpenkov | be6c329 | 2017-09-21 22:12:49 +0000 | [diff] [blame] | 122 | if 'CC' in os.environ: |
| 123 | Clang = os.environ['CC'] |
| 124 | else: |
George Karpenkov | bf92c44 | 2017-10-24 23:52:48 +0000 | [diff] [blame] | 125 | Clang = SATestUtils.which("clang", os.environ['PATH']) |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 126 | if not Clang: |
Serge Guelton | c0ebe77 | 2018-12-18 08:36:33 +0000 | [diff] [blame] | 127 | print("Error: cannot find 'clang' in PATH") |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 128 | sys.exit(1) |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 129 | |
Ted Kremenek | f9a539d | 2012-08-28 20:40:04 +0000 | [diff] [blame] | 130 | # Number of jobs. |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 131 | MaxJobs = int(math.ceil(multiprocessing.cpu_count() * 0.75)) |
Ted Kremenek | f9a539d | 2012-08-28 20:40:04 +0000 | [diff] [blame] | 132 | |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 133 | # Project map stores info about all the "registered" projects. |
| 134 | ProjectMapFile = "projectMap.csv" |
| 135 | |
| 136 | # Names of the project specific scripts. |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 137 | # The script that downloads the project. |
| 138 | DownloadScript = "download_project.sh" |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 139 | # The script that needs to be executed before the build can start. |
| 140 | CleanupScript = "cleanup_run_static_analyzer.sh" |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 141 | # This is a file containing commands for scan-build. |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 142 | BuildScript = "run_static_analyzer.cmd" |
| 143 | |
George Karpenkov | 5c23d6a | 2018-06-29 22:05:32 +0000 | [diff] [blame] | 144 | # A comment in a build script which disables wrapping. |
| 145 | NoPrefixCmd = "#NOPREFIX" |
| 146 | |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 147 | # The log file name. |
| 148 | LogFolderName = "Logs" |
| 149 | BuildLogName = "run_static_analyzer.log" |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 150 | # Summary file - contains the summary of the failures. Ex: This info can be be |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 151 | # displayed when buildbot detects a build failure. |
| 152 | NumOfFailuresInSummary = 10 |
| 153 | FailuresSummaryFileName = "failures.txt" |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 154 | |
| 155 | # The scan-build result directory. |
| 156 | SBOutputDirName = "ScanBuildResults" |
| 157 | SBOutputDirReferencePrefix = "Ref" |
| 158 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 159 | # The name of the directory storing the cached project source. If this |
| 160 | # directory does not exist, the download script will be executed. |
| 161 | # That script should create the "CachedSource" directory and download the |
| 162 | # project source into it. |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 163 | CachedSourceDirName = "CachedSource" |
| 164 | |
| 165 | # The name of the directory containing the source code that will be analyzed. |
| 166 | # Each time a project is analyzed, a fresh copy of its CachedSource directory |
| 167 | # will be copied to the PatchedSource directory and then the local patches |
| 168 | # in PatchfileName will be applied (if PatchfileName exists). |
| 169 | PatchedSourceDirName = "PatchedSource" |
| 170 | |
| 171 | # The name of the patchfile specifying any changes that should be applied |
| 172 | # to the CachedSource before analyzing. |
| 173 | PatchfileName = "changes_for_analyzer.patch" |
| 174 | |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 175 | # The list of checkers used during analyzes. |
Alp Toker | d473363 | 2013-12-05 04:47:09 +0000 | [diff] [blame] | 176 | # Currently, consists of all the non-experimental checkers, plus a few alpha |
Jordan Rose | 10ad081 | 2013-04-05 17:55:07 +0000 | [diff] [blame] | 177 | # checkers we don't want to regress on. |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 178 | Checkers = ",".join([ |
George Karpenkov | af76b4a | 2017-09-30 00:05:24 +0000 | [diff] [blame] | 179 | "alpha.unix.SimpleStream", |
| 180 | "alpha.security.taint", |
| 181 | "cplusplus.NewDeleteLeaks", |
| 182 | "core", |
| 183 | "cplusplus", |
| 184 | "deadcode", |
| 185 | "security", |
| 186 | "unix", |
| 187 | "osx", |
| 188 | "nullability" |
| 189 | ]) |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 190 | |
George Karpenkov | 7110581 | 2018-03-29 01:23:54 +0000 | [diff] [blame] | 191 | Verbose = 0 |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 192 | |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 193 | ############################################################################### |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 194 | # Test harness logic. |
George Karpenkov | 13d3748 | 2018-07-30 23:01:20 +0000 | [diff] [blame] | 195 | ############################################################################### |
Ted Kremenek | 42c1442 | 2012-08-28 20:40:02 +0000 | [diff] [blame] | 196 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 197 | |
Anna Zaks | 42a4463 | 2011-11-02 20:46:50 +0000 | [diff] [blame] | 198 | def runCleanupScript(Dir, PBuildLogFile): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 199 | """ |
| 200 | Run pre-processing script if any. |
| 201 | """ |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 202 | Cwd = os.path.join(Dir, PatchedSourceDirName) |
Anna Zaks | 42a4463 | 2011-11-02 20:46:50 +0000 | [diff] [blame] | 203 | ScriptPath = os.path.join(Dir, CleanupScript) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 204 | SATestUtils.runScript(ScriptPath, PBuildLogFile, Cwd, |
| 205 | Stdout=Local.stdout, Stderr=Local.stderr) |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 206 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 207 | |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 208 | def runDownloadScript(Dir, PBuildLogFile): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 209 | """ |
| 210 | Run the script to download the project, if it exists. |
| 211 | """ |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 212 | ScriptPath = os.path.join(Dir, DownloadScript) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 213 | SATestUtils.runScript(ScriptPath, PBuildLogFile, Dir, |
| 214 | Stdout=Local.stdout, Stderr=Local.stderr) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 215 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 216 | |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 217 | def downloadAndPatch(Dir, PBuildLogFile): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 218 | """ |
| 219 | Download the project and apply the local patchfile if it exists. |
| 220 | """ |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 221 | CachedSourceDirPath = os.path.join(Dir, CachedSourceDirName) |
| 222 | |
| 223 | # If the we don't already have the cached source, run the project's |
| 224 | # download script to download it. |
| 225 | if not os.path.exists(CachedSourceDirPath): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 226 | runDownloadScript(Dir, PBuildLogFile) |
| 227 | if not os.path.exists(CachedSourceDirPath): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 228 | Local.stderr.write("Error: '%s' not found after download.\n" % ( |
| 229 | CachedSourceDirPath)) |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 230 | exit(1) |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 231 | |
| 232 | PatchedSourceDirPath = os.path.join(Dir, PatchedSourceDirName) |
| 233 | |
| 234 | # Remove potentially stale patched source. |
| 235 | if os.path.exists(PatchedSourceDirPath): |
| 236 | shutil.rmtree(PatchedSourceDirPath) |
| 237 | |
| 238 | # Copy the cached source and apply any patches to the copy. |
| 239 | shutil.copytree(CachedSourceDirPath, PatchedSourceDirPath, symlinks=True) |
| 240 | applyPatch(Dir, PBuildLogFile) |
| 241 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 242 | |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 243 | def applyPatch(Dir, PBuildLogFile): |
| 244 | PatchfilePath = os.path.join(Dir, PatchfileName) |
| 245 | PatchedSourceDirPath = os.path.join(Dir, PatchedSourceDirName) |
| 246 | if not os.path.exists(PatchfilePath): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 247 | Local.stdout.write(" No local patches.\n") |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 248 | return |
| 249 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 250 | Local.stdout.write(" Applying patch.\n") |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 251 | try: |
Devin Coughlin | ab95cd2 | 2016-01-22 07:08:06 +0000 | [diff] [blame] | 252 | check_call("patch -p1 < '%s'" % (PatchfilePath), |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 253 | cwd=PatchedSourceDirPath, |
| 254 | stderr=PBuildLogFile, |
| 255 | stdout=PBuildLogFile, |
| 256 | shell=True) |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 257 | except: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 258 | Local.stderr.write("Error: Patch failed. See %s for details.\n" % ( |
| 259 | PBuildLogFile.name)) |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 260 | sys.exit(1) |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 261 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 262 | |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 263 | def generateAnalyzerConfig(Args): |
| 264 | Out = "serialize-stats=true,stable-report-filename=true" |
| 265 | if Args.extra_analyzer_config: |
| 266 | Out += "," + Args.extra_analyzer_config |
| 267 | return Out |
| 268 | |
| 269 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 270 | def runScanBuild(Args, Dir, SBOutputDir, PBuildLogFile): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 271 | """ |
| 272 | Build the project with scan-build by reading in the commands and |
| 273 | prefixing them with the scan-build options. |
| 274 | """ |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 275 | BuildScriptPath = os.path.join(Dir, BuildScript) |
| 276 | if not os.path.exists(BuildScriptPath): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 277 | Local.stderr.write( |
| 278 | "Error: build script is not defined: %s\n" % BuildScriptPath) |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 279 | sys.exit(1) |
Devin Coughlin | f3695c8 | 2015-09-16 01:52:32 +0000 | [diff] [blame] | 280 | |
| 281 | AllCheckers = Checkers |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 282 | if 'SA_ADDITIONAL_CHECKERS' in os.environ: |
Devin Coughlin | f3695c8 | 2015-09-16 01:52:32 +0000 | [diff] [blame] | 283 | AllCheckers = AllCheckers + ',' + os.environ['SA_ADDITIONAL_CHECKERS'] |
| 284 | |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 285 | # Run scan-build from within the patched source directory. |
| 286 | SBCwd = os.path.join(Dir, PatchedSourceDirName) |
| 287 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 288 | SBOptions = "--use-analyzer '%s' " % Clang |
Devin Coughlin | 86f61a9 | 2016-01-22 18:45:22 +0000 | [diff] [blame] | 289 | SBOptions += "-plist-html -o '%s' " % SBOutputDir |
Devin Coughlin | f3695c8 | 2015-09-16 01:52:32 +0000 | [diff] [blame] | 290 | SBOptions += "-enable-checker " + AllCheckers + " " |
Jordan Rose | b18179d | 2013-01-24 23:07:59 +0000 | [diff] [blame] | 291 | SBOptions += "--keep-empty " |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 292 | SBOptions += "-analyzer-config '%s' " % generateAnalyzerConfig(Args) |
Mikhail R. Gadelha | afc62b7 | 2018-06-27 14:39:41 +0000 | [diff] [blame] | 293 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 294 | # Always use ccc-analyze to ensure that we can locate the failures |
Anna Zaks | 7b4f8a4 | 2013-05-31 02:31:09 +0000 | [diff] [blame] | 295 | # directory. |
| 296 | SBOptions += "--override-compiler " |
George Karpenkov | 5c23d6a | 2018-06-29 22:05:32 +0000 | [diff] [blame] | 297 | ExtraEnv = {} |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 298 | try: |
| 299 | SBCommandFile = open(BuildScriptPath, "r") |
| 300 | SBPrefix = "scan-build " + SBOptions + " " |
| 301 | for Command in SBCommandFile: |
Jordan Rose | 7bd9186 | 2013-09-06 16:12:41 +0000 | [diff] [blame] | 302 | Command = Command.strip() |
Artem Dergachev | 11c8c2a | 2020-05-12 22:43:32 +0300 | [diff] [blame] | 303 | if len(Command) == 0: |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 304 | continue |
George Karpenkov | 5c23d6a | 2018-06-29 22:05:32 +0000 | [diff] [blame] | 305 | |
| 306 | # Custom analyzer invocation specified by project. |
| 307 | # Communicate required information using environment variables |
| 308 | # instead. |
| 309 | if Command == NoPrefixCmd: |
| 310 | SBPrefix = "" |
| 311 | ExtraEnv['OUTPUT'] = SBOutputDir |
George Karpenkov | 6e4ddf4 | 2018-07-02 17:10:40 +0000 | [diff] [blame] | 312 | ExtraEnv['CC'] = Clang |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 313 | ExtraEnv['ANALYZER_CONFIG'] = generateAnalyzerConfig(Args) |
George Karpenkov | 5c23d6a | 2018-06-29 22:05:32 +0000 | [diff] [blame] | 314 | continue |
| 315 | |
Artem Dergachev | 11c8c2a | 2020-05-12 22:43:32 +0300 | [diff] [blame] | 316 | if Command.startswith("#"): |
| 317 | continue |
| 318 | |
Ted Kremenek | f9a539d | 2012-08-28 20:40:04 +0000 | [diff] [blame] | 319 | # If using 'make', auto imply a -jX argument |
| 320 | # to speed up analysis. xcodebuild will |
| 321 | # automatically use the maximum number of cores. |
Jordan Rose | 64e4cf0 | 2012-11-26 19:59:57 +0000 | [diff] [blame] | 322 | if (Command.startswith("make ") or Command == "make") and \ |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 323 | "-j" not in Command: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 324 | Command += " -j%d" % MaxJobs |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 325 | SBCommand = SBPrefix + Command |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 326 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 327 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 328 | Local.stdout.write(" Executing: %s\n" % (SBCommand,)) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 329 | check_call(SBCommand, cwd=SBCwd, |
| 330 | stderr=PBuildLogFile, |
| 331 | stdout=PBuildLogFile, |
George Karpenkov | 5c23d6a | 2018-06-29 22:05:32 +0000 | [diff] [blame] | 332 | env=dict(os.environ, **ExtraEnv), |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 333 | shell=True) |
George Karpenkov | e58044d | 2017-10-27 22:39:54 +0000 | [diff] [blame] | 334 | except CalledProcessError: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 335 | Local.stderr.write("Error: scan-build failed. Its output was: \n") |
George Karpenkov | e58044d | 2017-10-27 22:39:54 +0000 | [diff] [blame] | 336 | PBuildLogFile.seek(0) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 337 | shutil.copyfileobj(PBuildLogFile, Local.stderr) |
George Karpenkov | e58044d | 2017-10-27 22:39:54 +0000 | [diff] [blame] | 338 | sys.exit(1) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 339 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 340 | |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 341 | def runAnalyzePreprocessed(Args, Dir, SBOutputDir, Mode): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 342 | """ |
| 343 | Run analysis on a set of preprocessed files. |
| 344 | """ |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 345 | if os.path.exists(os.path.join(Dir, BuildScript)): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 346 | Local.stderr.write( |
| 347 | "Error: The preprocessed files project should not contain %s\n" % ( |
| 348 | BuildScript)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 349 | raise Exception() |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 350 | |
Artem Dergachev | e8b29c0 | 2019-05-29 18:49:31 +0000 | [diff] [blame] | 351 | CmdPrefix = Clang + " --analyze " |
Devin Coughlin | bace032 | 2015-09-14 21:22:24 +0000 | [diff] [blame] | 352 | |
Artem Dergachev | e8b29c0 | 2019-05-29 18:49:31 +0000 | [diff] [blame] | 353 | CmdPrefix += "--analyzer-output plist " |
| 354 | CmdPrefix += " -Xclang -analyzer-checker=" + Checkers |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 355 | CmdPrefix += " -fcxx-exceptions -fblocks " |
Artem Dergachev | e8b29c0 | 2019-05-29 18:49:31 +0000 | [diff] [blame] | 356 | CmdPrefix += " -Xclang -analyzer-config -Xclang %s "\ |
| 357 | % generateAnalyzerConfig(Args) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 358 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 359 | if (Mode == 2): |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 360 | CmdPrefix += "-std=c++11 " |
| 361 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 362 | PlistPath = os.path.join(Dir, SBOutputDir, "date") |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 363 | FailPath = os.path.join(PlistPath, "failures") |
| 364 | os.makedirs(FailPath) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 365 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 366 | for FullFileName in glob.glob(Dir + "/*"): |
| 367 | FileName = os.path.basename(FullFileName) |
| 368 | Failed = False |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 369 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 370 | # Only run the analyzes on supported files. |
George Karpenkov | bf92c44 | 2017-10-24 23:52:48 +0000 | [diff] [blame] | 371 | if SATestUtils.hasNoExtension(FileName): |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 372 | continue |
George Karpenkov | bf92c44 | 2017-10-24 23:52:48 +0000 | [diff] [blame] | 373 | if not SATestUtils.isValidSingleInputFile(FileName): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 374 | Local.stderr.write( |
| 375 | "Error: Invalid single input file %s.\n" % (FullFileName,)) |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 376 | raise Exception() |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 377 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 378 | # Build and call the analyzer command. |
Devin Coughlin | ab95cd2 | 2016-01-22 07:08:06 +0000 | [diff] [blame] | 379 | OutputOption = "-o '%s.plist' " % os.path.join(PlistPath, FileName) |
| 380 | Command = CmdPrefix + OutputOption + ("'%s'" % FileName) |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 381 | LogFile = open(os.path.join(FailPath, FileName + ".stderr.txt"), "w+") |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 382 | try: |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 383 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 384 | Local.stdout.write(" Executing: %s\n" % (Command,)) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 385 | check_call(Command, cwd=Dir, stderr=LogFile, |
| 386 | stdout=LogFile, |
| 387 | shell=True) |
Serge Guelton | 3de4108 | 2018-12-03 12:11:21 +0000 | [diff] [blame] | 388 | except CalledProcessError as e: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 389 | Local.stderr.write("Error: Analyzes of %s failed. " |
| 390 | "See %s for details." |
| 391 | "Error code %d.\n" % ( |
| 392 | FullFileName, LogFile.name, e.returncode)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 393 | Failed = True |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 394 | finally: |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 395 | LogFile.close() |
| 396 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 397 | # If command did not fail, erase the log file. |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 398 | if not Failed: |
| 399 | os.remove(LogFile.name) |
| 400 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 401 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 402 | def getBuildLogPath(SBOutputDir): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 403 | return os.path.join(SBOutputDir, LogFolderName, BuildLogName) |
| 404 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 405 | |
| 406 | def removeLogFile(SBOutputDir): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 407 | BuildLogPath = getBuildLogPath(SBOutputDir) |
| 408 | # Clean up the log file. |
| 409 | if (os.path.exists(BuildLogPath)): |
| 410 | RmCommand = "rm '%s'" % BuildLogPath |
| 411 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 412 | Local.stdout.write(" Executing: %s\n" % (RmCommand,)) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 413 | check_call(RmCommand, shell=True) |
| 414 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 415 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 416 | def buildProject(Args, Dir, SBOutputDir, ProjectBuildMode, IsReferenceBuild): |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 417 | TBegin = time.time() |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 418 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 419 | BuildLogPath = getBuildLogPath(SBOutputDir) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 420 | Local.stdout.write("Log file: %s\n" % (BuildLogPath,)) |
| 421 | Local.stdout.write("Output directory: %s\n" % (SBOutputDir, )) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 422 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 423 | removeLogFile(SBOutputDir) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 424 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 425 | # Clean up scan build results. |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 426 | if (os.path.exists(SBOutputDir)): |
Devin Coughlin | ab95cd2 | 2016-01-22 07:08:06 +0000 | [diff] [blame] | 427 | RmCommand = "rm -r '%s'" % SBOutputDir |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 428 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 429 | Local.stdout.write(" Executing: %s\n" % (RmCommand,)) |
| 430 | check_call(RmCommand, shell=True, stdout=Local.stdout, |
| 431 | stderr=Local.stderr) |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 432 | assert(not os.path.exists(SBOutputDir)) |
| 433 | os.makedirs(os.path.join(SBOutputDir, LogFolderName)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 434 | |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 435 | # Build and analyze the project. |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 436 | with open(BuildLogPath, "w+") as PBuildLogFile: |
Anna Zaks | a2f970b | 2012-09-06 23:30:27 +0000 | [diff] [blame] | 437 | if (ProjectBuildMode == 1): |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 438 | downloadAndPatch(Dir, PBuildLogFile) |
| 439 | runCleanupScript(Dir, PBuildLogFile) |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 440 | runScanBuild(Args, Dir, SBOutputDir, PBuildLogFile) |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 441 | else: |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 442 | runAnalyzePreprocessed(Args, Dir, SBOutputDir, ProjectBuildMode) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 443 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 444 | if IsReferenceBuild: |
Anna Zaks | 42a4463 | 2011-11-02 20:46:50 +0000 | [diff] [blame] | 445 | runCleanupScript(Dir, PBuildLogFile) |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 446 | normalizeReferenceResults(Dir, SBOutputDir, ProjectBuildMode) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 447 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 448 | Local.stdout.write("Build complete (time: %.2f). " |
| 449 | "See the log for more details: %s\n" % ( |
| 450 | (time.time() - TBegin), BuildLogPath)) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 451 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 452 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 453 | def normalizeReferenceResults(Dir, SBOutputDir, ProjectBuildMode): |
| 454 | """ |
| 455 | Make the absolute paths relative in the reference results. |
| 456 | """ |
| 457 | for (DirPath, Dirnames, Filenames) in os.walk(SBOutputDir): |
| 458 | for F in Filenames: |
| 459 | if (not F.endswith('plist')): |
| 460 | continue |
| 461 | Plist = os.path.join(DirPath, F) |
| 462 | Data = plistlib.readPlist(Plist) |
| 463 | PathPrefix = Dir |
| 464 | if (ProjectBuildMode == 1): |
| 465 | PathPrefix = os.path.join(Dir, PatchedSourceDirName) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 466 | Paths = [SourceFile[len(PathPrefix) + 1:] |
| 467 | if SourceFile.startswith(PathPrefix) |
| 468 | else SourceFile for SourceFile in Data['files']] |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 469 | Data['files'] = Paths |
George Karpenkov | 318cd1f | 2017-10-24 23:52:46 +0000 | [diff] [blame] | 470 | |
| 471 | # Remove transient fields which change from run to run. |
| 472 | for Diag in Data['diagnostics']: |
| 473 | if 'HTMLDiagnostics_files' in Diag: |
| 474 | Diag.pop('HTMLDiagnostics_files') |
| 475 | if 'clang_version' in Data: |
| 476 | Data.pop('clang_version') |
| 477 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 478 | plistlib.writePlist(Data, Plist) |
| 479 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 480 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 481 | def CleanUpEmptyPlists(SBOutputDir): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 482 | """ |
| 483 | A plist file is created for each call to the analyzer(each source file). |
| 484 | We are only interested on the once that have bug reports, |
| 485 | so delete the rest. |
| 486 | """ |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 487 | for F in glob.glob(SBOutputDir + "/*/*.plist"): |
| 488 | P = os.path.join(SBOutputDir, F) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 489 | |
Artem Dergachev | 8cbd3f4 | 2020-05-13 14:06:44 +0300 | [diff] [blame] | 490 | try: |
| 491 | Data = plistlib.readPlist(P) |
| 492 | # Delete empty reports. |
| 493 | if not Data['files']: |
| 494 | os.remove(P) |
| 495 | continue |
Artem Dergachev | 1a3b801 | 2020-05-15 14:27:30 +0300 | [diff] [blame^] | 496 | except plistlib.InvalidFileException as e: |
Artem Dergachev | 8cbd3f4 | 2020-05-13 14:06:44 +0300 | [diff] [blame] | 497 | print('Error parsing plist file %s: %s' % (P, str(e))) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 498 | continue |
| 499 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 500 | |
George Karpenkov | 3c128cb | 2017-10-30 19:40:33 +0000 | [diff] [blame] | 501 | def CleanUpEmptyFolders(SBOutputDir): |
| 502 | """ |
| 503 | Remove empty folders from results, as git would not store them. |
| 504 | """ |
| 505 | Subfolders = glob.glob(SBOutputDir + "/*") |
| 506 | for Folder in Subfolders: |
| 507 | if not os.listdir(Folder): |
| 508 | os.removedirs(Folder) |
| 509 | |
| 510 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 511 | def checkBuild(SBOutputDir): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 512 | """ |
| 513 | Given the scan-build output directory, checks if the build failed |
| 514 | (by searching for the failures directories). If there are failures, it |
| 515 | creates a summary file in the output directory. |
| 516 | |
| 517 | """ |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 518 | # Check if there are failures. |
| 519 | Failures = glob.glob(SBOutputDir + "/*/failures/*.stderr.txt") |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 520 | TotalFailed = len(Failures) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 521 | if TotalFailed == 0: |
Jordan Rose | 9858b12 | 2012-08-31 00:36:30 +0000 | [diff] [blame] | 522 | CleanUpEmptyPlists(SBOutputDir) |
George Karpenkov | 3c128cb | 2017-10-30 19:40:33 +0000 | [diff] [blame] | 523 | CleanUpEmptyFolders(SBOutputDir) |
Jordan Rose | 9858b12 | 2012-08-31 00:36:30 +0000 | [diff] [blame] | 524 | Plists = glob.glob(SBOutputDir + "/*/*.plist") |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 525 | Local.stdout.write( |
| 526 | "Number of bug reports (non-empty plist files) produced: %d\n" % |
| 527 | len(Plists)) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 528 | return |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 529 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 530 | Local.stderr.write("Error: analysis failed.\n") |
| 531 | Local.stderr.write("Total of %d failures discovered.\n" % TotalFailed) |
George Karpenkov | ff555ce | 2017-10-26 19:00:22 +0000 | [diff] [blame] | 532 | if TotalFailed > NumOfFailuresInSummary: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 533 | Local.stderr.write( |
| 534 | "See the first %d below.\n" % NumOfFailuresInSummary) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 535 | # TODO: Add a line "See the results folder for more." |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 536 | |
George Karpenkov | ff555ce | 2017-10-26 19:00:22 +0000 | [diff] [blame] | 537 | Idx = 0 |
| 538 | for FailLogPathI in Failures: |
| 539 | if Idx >= NumOfFailuresInSummary: |
| 540 | break |
| 541 | Idx += 1 |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 542 | Local.stderr.write("\n-- Error #%d -----------\n" % Idx) |
George Karpenkov | ff555ce | 2017-10-26 19:00:22 +0000 | [diff] [blame] | 543 | with open(FailLogPathI, "r") as FailLogI: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 544 | shutil.copyfileobj(FailLogI, Local.stdout) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 545 | |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 546 | sys.exit(1) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 547 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 548 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 549 | def runCmpResults(Dir, Strictness=0): |
| 550 | """ |
| 551 | Compare the warnings produced by scan-build. |
| 552 | Strictness defines the success criteria for the test: |
| 553 | 0 - success if there are no crashes or analyzer failure. |
| 554 | 1 - success if there are no difference in the number of reported bugs. |
| 555 | 2 - success if all the bug reports are identical. |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 556 | |
| 557 | :return success: Whether tests pass according to the Strictness |
| 558 | criteria. |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 559 | """ |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 560 | TestsPassed = True |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 561 | TBegin = time.time() |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 562 | |
| 563 | RefDir = os.path.join(Dir, SBOutputDirReferencePrefix + SBOutputDirName) |
| 564 | NewDir = os.path.join(Dir, SBOutputDirName) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 565 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 566 | # We have to go one level down the directory tree. |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 567 | RefList = glob.glob(RefDir + "/*") |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 568 | NewList = glob.glob(NewDir + "/*") |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 569 | |
Jordan Rose | c7b992e | 2013-06-10 19:34:30 +0000 | [diff] [blame] | 570 | # Log folders are also located in the results dir, so ignore them. |
| 571 | RefLogDir = os.path.join(RefDir, LogFolderName) |
| 572 | if RefLogDir in RefList: |
| 573 | RefList.remove(RefLogDir) |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 574 | NewList.remove(os.path.join(NewDir, LogFolderName)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 575 | |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 576 | if len(RefList) != len(NewList): |
Serge Guelton | c0ebe77 | 2018-12-18 08:36:33 +0000 | [diff] [blame] | 577 | print("Mismatch in number of results folders: %s vs %s" % ( |
| 578 | RefList, NewList)) |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 579 | sys.exit(1) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 580 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 581 | # There might be more then one folder underneath - one per each scan-build |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 582 | # command (Ex: one for configure and one for make). |
| 583 | if (len(RefList) > 1): |
| 584 | # Assume that the corresponding folders have the same names. |
| 585 | RefList.sort() |
| 586 | NewList.sort() |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 587 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 588 | # Iterate and find the differences. |
Anna Zaks | 767d356 | 2011-11-08 19:56:31 +0000 | [diff] [blame] | 589 | NumDiffs = 0 |
Serge Guelton | d458974 | 2018-12-18 16:04:21 +0000 | [diff] [blame] | 590 | for P in zip(RefList, NewList): |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 591 | RefDir = P[0] |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 592 | NewDir = P[1] |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 593 | |
| 594 | assert(RefDir != NewDir) |
| 595 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 596 | Local.stdout.write(" Comparing Results: %s %s\n" % ( |
| 597 | RefDir, NewDir)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 598 | |
Devin Coughlin | 2cb767d | 2015-11-07 18:27:35 +0000 | [diff] [blame] | 599 | PatchedSourceDirPath = os.path.join(Dir, PatchedSourceDirName) |
George Karpenkov | fc782a3 | 2018-02-09 18:39:47 +0000 | [diff] [blame] | 600 | Opts, Args = CmpRuns.generate_option_parser().parse_args( |
George Karpenkov | 192d9a1 | 2018-02-12 22:13:01 +0000 | [diff] [blame] | 601 | ["--rootA", "", "--rootB", PatchedSourceDirPath]) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 602 | # Scan the results, delete empty plist files. |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 603 | NumDiffs, ReportsInRef, ReportsInNew = \ |
George Karpenkov | b7120c9 | 2018-02-13 23:36:01 +0000 | [diff] [blame] | 604 | CmpRuns.dumpScanBuildResultsDiff(RefDir, NewDir, Opts, |
| 605 | deleteEmpty=False, |
| 606 | Stdout=Local.stdout) |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 607 | if (NumDiffs > 0): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 608 | Local.stdout.write("Warning: %s differences in diagnostics.\n" |
| 609 | % NumDiffs) |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 610 | if Strictness >= 2 and NumDiffs > 0: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 611 | Local.stdout.write("Error: Diffs found in strict mode (2).\n") |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 612 | TestsPassed = False |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 613 | elif Strictness >= 1 and ReportsInRef != ReportsInNew: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 614 | Local.stdout.write("Error: The number of results are different " + |
| 615 | " strict mode (1).\n") |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 616 | TestsPassed = False |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 617 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 618 | Local.stdout.write("Diagnostic comparison complete (time: %.2f).\n" % ( |
| 619 | time.time() - TBegin)) |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 620 | return TestsPassed |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 621 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 622 | |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 623 | def cleanupReferenceResults(SBOutputDir): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 624 | """ |
| 625 | Delete html, css, and js files from reference results. These can |
| 626 | include multiple copies of the benchmark source and so get very large. |
| 627 | """ |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 628 | Extensions = ["html", "css", "js"] |
| 629 | for E in Extensions: |
| 630 | for F in glob.glob("%s/*/*.%s" % (SBOutputDir, E)): |
| 631 | P = os.path.join(SBOutputDir, F) |
| 632 | RmCommand = "rm '%s'" % P |
| 633 | check_call(RmCommand, shell=True) |
| 634 | |
| 635 | # Remove the log file. It leaks absolute path names. |
| 636 | removeLogFile(SBOutputDir) |
| 637 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 638 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 639 | class TestProjectThread(threading.Thread): |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 640 | def __init__(self, Args, TasksQueue, ResultsDiffer, FailureFlag): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 641 | """ |
| 642 | :param ResultsDiffer: Used to signify that results differ from |
| 643 | the canonical ones. |
| 644 | :param FailureFlag: Used to signify a failure during the run. |
| 645 | """ |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 646 | self.Args = Args |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 647 | self.TasksQueue = TasksQueue |
| 648 | self.ResultsDiffer = ResultsDiffer |
| 649 | self.FailureFlag = FailureFlag |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 650 | super().__init__() |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 651 | |
| 652 | # Needed to gracefully handle interrupts with Ctrl-C |
| 653 | self.daemon = True |
| 654 | |
| 655 | def run(self): |
| 656 | while not self.TasksQueue.empty(): |
| 657 | try: |
| 658 | ProjArgs = self.TasksQueue.get() |
| 659 | Logger = logging.getLogger(ProjArgs[0]) |
| 660 | Local.stdout = StreamToLogger(Logger, logging.INFO) |
| 661 | Local.stderr = StreamToLogger(Logger, logging.ERROR) |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 662 | if not testProject(Args, *ProjArgs): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 663 | self.ResultsDiffer.set() |
| 664 | self.TasksQueue.task_done() |
| 665 | except: |
| 666 | self.FailureFlag.set() |
| 667 | raise |
| 668 | |
| 669 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 670 | def testProject(Args, ID, ProjectBuildMode, IsReferenceBuild=False, Strictness=0): |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 671 | """ |
| 672 | Test a given project. |
| 673 | :return TestsPassed: Whether tests have passed according |
| 674 | to the :param Strictness: criteria. |
| 675 | """ |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 676 | Local.stdout.write(" \n\n--- Building project %s\n" % (ID,)) |
Anna Zaks | 4720a73 | 2011-11-05 05:20:48 +0000 | [diff] [blame] | 677 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 678 | TBegin = time.time() |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 679 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 680 | Dir = getProjectDir(ID) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 681 | if Verbose == 1: |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 682 | Local.stdout.write(" Build directory: %s.\n" % (Dir,)) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 683 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 684 | # Set the build results directory. |
Jordan Rose | 01ac572 | 2012-06-01 16:24:38 +0000 | [diff] [blame] | 685 | RelOutputDir = getSBOutputDirName(IsReferenceBuild) |
Anna Zaks | 4c1ef976 | 2012-02-03 06:35:23 +0000 | [diff] [blame] | 686 | SBOutputDir = os.path.join(Dir, RelOutputDir) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 687 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 688 | buildProject(Args, Dir, SBOutputDir, ProjectBuildMode, IsReferenceBuild) |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 689 | |
| 690 | checkBuild(SBOutputDir) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 691 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 692 | if IsReferenceBuild: |
Devin Coughlin | 9ea8033 | 2016-01-23 01:09:07 +0000 | [diff] [blame] | 693 | cleanupReferenceResults(SBOutputDir) |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 694 | TestsPassed = True |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 695 | else: |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 696 | TestsPassed = runCmpResults(Dir, Strictness) |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 697 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 698 | Local.stdout.write("Completed tests for project %s (time: %.2f).\n" % ( |
| 699 | ID, (time.time() - TBegin))) |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 700 | return TestsPassed |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 701 | |
Ted Kremenek | 3a0678e | 2015-09-08 03:50:52 +0000 | [diff] [blame] | 702 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 703 | def projectFileHandler(): |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 704 | return open(getProjectMapPath(), "r") |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 705 | |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 706 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 707 | def iterateOverProjects(PMapFile): |
| 708 | """ |
| 709 | Iterate over all projects defined in the project file handler `PMapFile` |
| 710 | from the start. |
| 711 | """ |
| 712 | PMapFile.seek(0) |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 713 | for ProjectInfo in csv.reader(PMapFile): |
| 714 | if (SATestUtils.isCommentCSVLine(ProjectInfo)): |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 715 | continue |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 716 | yield ProjectInfo |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 717 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 718 | |
| 719 | def validateProjectFile(PMapFile): |
| 720 | """ |
| 721 | Validate project file. |
| 722 | """ |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 723 | for ProjectInfo in iterateOverProjects(PMapFile): |
| 724 | if len(ProjectInfo) != 2: |
Serge Guelton | c0ebe77 | 2018-12-18 08:36:33 +0000 | [diff] [blame] | 725 | print("Error: Rows in the ProjectMapFile should have 2 entries.") |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 726 | raise Exception() |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 727 | if ProjectInfo[1] not in ('0', '1', '2'): |
| 728 | print("Error: Second entry in the ProjectMapFile should be 0" |
Serge Guelton | c0ebe77 | 2018-12-18 08:36:33 +0000 | [diff] [blame] | 729 | " (single file), 1 (project), or 2(single file c++11).") |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 730 | raise Exception() |
| 731 | |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 732 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 733 | def singleThreadedTestAll(Args, ProjectsToTest): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 734 | """ |
| 735 | Run all projects. |
| 736 | :return: whether tests have passed. |
| 737 | """ |
| 738 | Success = True |
| 739 | for ProjArgs in ProjectsToTest: |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 740 | Success &= testProject(Args, *ProjArgs) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 741 | return Success |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 742 | |
Valeriy Savchenko | c98872e | 2020-05-14 13:31:01 +0300 | [diff] [blame] | 743 | |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 744 | def multiThreadedTestAll(Args, ProjectsToTest, Jobs): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 745 | """ |
| 746 | Run each project in a separate thread. |
| 747 | |
| 748 | This is OK despite GIL, as testing is blocked |
| 749 | on launching external processes. |
| 750 | |
| 751 | :return: whether tests have passed. |
| 752 | """ |
Serge Guelton | 1f88dc5 | 2018-12-13 07:44:19 +0000 | [diff] [blame] | 753 | TasksQueue = queue.Queue() |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 754 | |
| 755 | for ProjArgs in ProjectsToTest: |
| 756 | TasksQueue.put(ProjArgs) |
| 757 | |
| 758 | ResultsDiffer = threading.Event() |
| 759 | FailureFlag = threading.Event() |
| 760 | |
| 761 | for i in range(Jobs): |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 762 | T = TestProjectThread(Args, TasksQueue, ResultsDiffer, FailureFlag) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 763 | T.start() |
| 764 | |
| 765 | # Required to handle Ctrl-C gracefully. |
| 766 | while TasksQueue.unfinished_tasks: |
| 767 | time.sleep(0.1) # Seconds. |
| 768 | if FailureFlag.is_set(): |
| 769 | Local.stderr.write("Test runner crashed\n") |
| 770 | sys.exit(1) |
| 771 | return not ResultsDiffer.is_set() |
| 772 | |
| 773 | |
| 774 | def testAll(Args): |
| 775 | ProjectsToTest = [] |
| 776 | |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 777 | with projectFileHandler() as PMapFile: |
| 778 | validateProjectFile(PMapFile) |
Anna Zaks | 4c1ef976 | 2012-02-03 06:35:23 +0000 | [diff] [blame] | 779 | |
Anna Zaks | 4c1ef976 | 2012-02-03 06:35:23 +0000 | [diff] [blame] | 780 | # Test the projects. |
George Karpenkov | 3abfc3b | 2017-09-22 01:41:16 +0000 | [diff] [blame] | 781 | for (ProjName, ProjBuildMode) in iterateOverProjects(PMapFile): |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 782 | ProjectsToTest.append((ProjName, |
| 783 | int(ProjBuildMode), |
| 784 | Args.regenerate, |
| 785 | Args.strictness)) |
| 786 | if Args.jobs <= 1: |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 787 | return singleThreadedTestAll(Args, ProjectsToTest) |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 788 | else: |
George Karpenkov | 47e5493 | 2018-09-27 01:10:59 +0000 | [diff] [blame] | 789 | return multiThreadedTestAll(Args, ProjectsToTest, Args.jobs) |
Anna Zaks | 4c1ef976 | 2012-02-03 06:35:23 +0000 | [diff] [blame] | 790 | |
Anna Zaks | 4c1ef976 | 2012-02-03 06:35:23 +0000 | [diff] [blame] | 791 | |
Anna Zaks | f0c4116 | 2011-10-06 23:26:27 +0000 | [diff] [blame] | 792 | if __name__ == '__main__': |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 793 | # Parse command line arguments. |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 794 | Parser = argparse.ArgumentParser( |
| 795 | description='Test the Clang Static Analyzer.') |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 796 | Parser.add_argument('--strictness', dest='strictness', type=int, default=0, |
George Karpenkov | a807660 | 2017-10-02 17:59:12 +0000 | [diff] [blame] | 797 | help='0 to fail on runtime errors, 1 to fail when the \ |
| 798 | number of found bugs are different from the \ |
| 799 | reference, 2 to fail on any difference from the \ |
| 800 | reference. Default is 0.') |
| 801 | Parser.add_argument('-r', dest='regenerate', action='store_true', |
| 802 | default=False, help='Regenerate reference output.') |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 803 | Parser.add_argument('-j', '--jobs', dest='jobs', type=int, |
| 804 | default=0, |
| 805 | help='Number of projects to test concurrently') |
George Karpenkov | ac98683 | 2018-10-02 21:19:23 +0000 | [diff] [blame] | 806 | Parser.add_argument('--extra-analyzer-config', dest='extra_analyzer_config', |
| 807 | type=str, |
| 808 | default="", |
| 809 | help="Arguments passed to to -analyzer-config") |
Gabor Horvath | 93fde94 | 2015-06-30 15:31:17 +0000 | [diff] [blame] | 810 | Args = Parser.parse_args() |
| 811 | |
George Karpenkov | f37d3a5 | 2018-02-08 21:22:42 +0000 | [diff] [blame] | 812 | TestsPassed = testAll(Args) |
George Karpenkov | 1b51cbd | 2017-10-05 17:32:06 +0000 | [diff] [blame] | 813 | if not TestsPassed: |
Serge Guelton | c0ebe77 | 2018-12-18 08:36:33 +0000 | [diff] [blame] | 814 | print("ERROR: Tests failed.") |
George Karpenkov | 65839bd | 2017-10-26 01:13:22 +0000 | [diff] [blame] | 815 | sys.exit(42) |