mirror of https://github.com/google/oss-fuzz.git
bisector: Properly detect MSan issues as well. (#3762)
Adopt a similar start/end marker approach to CF.
This commit is contained in:
parent
2d972c9162
commit
b16fcfc903
|
@ -47,8 +47,12 @@ import utils
|
|||
|
||||
Result = collections.namedtuple('Result', ['repo_url', 'commit'])
|
||||
|
||||
CRASH_MARKERS = [
|
||||
START_MARKERS = [
|
||||
'==ERROR',
|
||||
'==WARNING',
|
||||
]
|
||||
|
||||
END_MARKERS = [
|
||||
'SUMMARY:',
|
||||
]
|
||||
|
||||
|
@ -157,8 +161,12 @@ def _check_for_crash(project_name, fuzz_target, test_case_path):
|
|||
|
||||
logging.info('stdout =\n%s', out)
|
||||
logging.info('stderr =\n%s', err)
|
||||
|
||||
# pylint: disable=unsupported-membership-test
|
||||
return all(marker in out or marker in err for marker in CRASH_MARKERS)
|
||||
has_start_marker = any(
|
||||
marker in out or marker in err for marker in START_MARKERS)
|
||||
has_end_marker = any(marker in out or marker in err for marker in END_MARKERS)
|
||||
return has_start_marker and has_end_marker
|
||||
|
||||
|
||||
# pylint: disable=too-many-locals
|
||||
|
|
Loading…
Reference in New Issue