mirror of https://github.com/google/oss-fuzz.git
315 lines
11 KiB
Python
315 lines
11 KiB
Python
# Copyright 2019 Google LLC
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
"""Uses bisection to determine which commit a bug was introduced and fixed.
|
|
This module takes a high and a low commit SHA, a repo name, and a bug.
|
|
The module bisects the high and low commit SHA searching for the location
|
|
where the bug was introduced. It also looks for where the bug was fixed.
|
|
This is done with the following steps:
|
|
|
|
|
|
NOTE: Needs to be run from root of the OSS-Fuzz source checkout.
|
|
|
|
Typical usage example:
|
|
python3 infra/bisector.py
|
|
--old_commit 1e403e9259a1abedf108ab86f711ba52c907226d
|
|
--new_commit f79be4f2330f4b89ea2f42e1c44ca998c59a0c0f
|
|
--fuzz_target rules_fuzzer
|
|
--project_name yara
|
|
--testcase infra/yara_testcase
|
|
--sanitizer address
|
|
"""
|
|
|
|
import argparse
|
|
import collections
|
|
import logging
|
|
import os
|
|
import sys
|
|
import tempfile
|
|
|
|
import build_specified_commit
|
|
import helper
|
|
import repo_manager
|
|
import utils
|
|
|
|
Result = collections.namedtuple('Result', ['repo_url', 'commit'])
|
|
|
|
START_MARKERS = [
|
|
'==ERROR',
|
|
'==WARNING',
|
|
]
|
|
|
|
END_MARKERS = [
|
|
'SUMMARY:',
|
|
]
|
|
|
|
DEDUP_TOKEN_MARKER = 'DEDUP_TOKEN:'
|
|
|
|
|
|
class BisectError(Exception):
|
|
"""Bisection error."""
|
|
|
|
def __init__(self, message, repo_url):
|
|
super().__init__(message)
|
|
self.repo_url = repo_url
|
|
|
|
|
|
def main():
|
|
"""Finds the commit SHA where an error was initally introduced."""
|
|
logging.getLogger().setLevel(logging.INFO)
|
|
utils.chdir_to_root()
|
|
parser = argparse.ArgumentParser(
|
|
description='git bisection for finding introduction of bugs')
|
|
|
|
parser.add_argument('--project_name',
|
|
help='The name of the project where the bug occurred.',
|
|
required=True)
|
|
parser.add_argument('--new_commit',
|
|
help='The newest commit SHA to be bisected.',
|
|
required=True)
|
|
parser.add_argument('--old_commit',
|
|
help='The oldest commit SHA to be bisected.',
|
|
required=True)
|
|
parser.add_argument('--fuzz_target',
|
|
help='The name of the fuzzer to be built.',
|
|
required=True)
|
|
parser.add_argument('--test_case_path',
|
|
help='The path to test case.',
|
|
required=True)
|
|
parser.add_argument('--engine',
|
|
help='The default is "libfuzzer".',
|
|
default='libfuzzer')
|
|
parser.add_argument('--sanitizer',
|
|
default='address',
|
|
help='The default is "address".')
|
|
parser.add_argument('--type',
|
|
choices=['regressed', 'fixed'],
|
|
help='The bisection type.',
|
|
required=True)
|
|
parser.add_argument('--architecture', default='x86_64')
|
|
args = parser.parse_args()
|
|
|
|
build_data = build_specified_commit.BuildData(project_name=args.project_name,
|
|
engine=args.engine,
|
|
sanitizer=args.sanitizer,
|
|
architecture=args.architecture)
|
|
|
|
result = bisect(args.type, args.old_commit, args.new_commit,
|
|
args.test_case_path, args.fuzz_target, build_data)
|
|
if not result.commit:
|
|
logging.error('No error was found in commit range %s:%s', args.old_commit,
|
|
args.new_commit)
|
|
return 1
|
|
if result.commit == args.old_commit:
|
|
logging.error(
|
|
'Bisection Error: Both the first and the last commits in'
|
|
'the given range have the same behavior, bisection is not possible. ')
|
|
return 1
|
|
print('Error was introduced at commit %s' % result.commit)
|
|
return 0
|
|
|
|
|
|
def _get_dedup_token(output):
|
|
"""Get dedup token."""
|
|
for line in output.splitlines():
|
|
token_location = line.find(DEDUP_TOKEN_MARKER)
|
|
if token_location == -1:
|
|
continue
|
|
|
|
return line[token_location + len(DEDUP_TOKEN_MARKER):].strip()
|
|
|
|
return None
|
|
|
|
|
|
def _check_for_crash(project_name, fuzz_target, testcase_path):
|
|
"""Check for crash."""
|
|
|
|
def docker_run(args):
|
|
command = ['docker', 'run', '--rm', '--privileged']
|
|
if sys.stdin.isatty():
|
|
command.append('-i')
|
|
|
|
return utils.execute(command + args)
|
|
|
|
logging.info('Checking for crash')
|
|
out, err, return_code = helper.reproduce_impl(
|
|
project=helper.Project(project_name),
|
|
fuzzer_name=fuzz_target,
|
|
valgrind=False,
|
|
env_to_add=[],
|
|
fuzzer_args=[],
|
|
testcase_path=testcase_path,
|
|
run_function=docker_run,
|
|
err_result=(None, None, None))
|
|
if return_code is None:
|
|
return None
|
|
|
|
logging.info('stdout =\n%s', out)
|
|
logging.info('stderr =\n%s', err)
|
|
|
|
# pylint: disable=unsupported-membership-test
|
|
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)
|
|
if not has_start_marker or not has_end_marker:
|
|
return None
|
|
|
|
return _get_dedup_token(out + err)
|
|
|
|
|
|
# pylint: disable=too-many-locals
|
|
# pylint: disable=too-many-arguments
|
|
# pylint: disable=too-many-statements
|
|
def _bisect(bisect_type, old_commit, new_commit, testcase_path, fuzz_target,
|
|
build_data):
|
|
"""Perform the bisect."""
|
|
# pylint: disable=too-many-branches
|
|
base_builder_repo = build_specified_commit.load_base_builder_repo()
|
|
|
|
with tempfile.TemporaryDirectory() as tmp_dir:
|
|
repo_url, repo_path = build_specified_commit.detect_main_repo(
|
|
build_data.project_name, commit=new_commit)
|
|
if not repo_url or not repo_path:
|
|
raise ValueError('Main git repo can not be determined.')
|
|
|
|
if old_commit == new_commit:
|
|
raise BisectError('old_commit is the same as new_commit', repo_url)
|
|
|
|
# Copy /src from the built Docker container to ensure all dependencies
|
|
# exist. This will be mounted when running them.
|
|
host_src_dir = build_specified_commit.copy_src_from_docker(
|
|
build_data.project_name, tmp_dir)
|
|
|
|
bisect_repo_manager = repo_manager.RepoManager(
|
|
os.path.join(host_src_dir, os.path.basename(repo_path)))
|
|
bisect_repo_manager.fetch_all_remotes()
|
|
|
|
commit_list = bisect_repo_manager.get_commit_list(new_commit, old_commit)
|
|
|
|
old_idx = len(commit_list) - 1
|
|
new_idx = 0
|
|
logging.info('Testing against new_commit (%s)', commit_list[new_idx])
|
|
if not build_specified_commit.build_fuzzers_from_commit(
|
|
commit_list[new_idx],
|
|
bisect_repo_manager,
|
|
host_src_dir,
|
|
build_data,
|
|
base_builder_repo=base_builder_repo):
|
|
raise BisectError('Failed to build new_commit', repo_url)
|
|
|
|
if bisect_type == 'fixed':
|
|
should_crash = False
|
|
elif bisect_type == 'regressed':
|
|
should_crash = True
|
|
else:
|
|
raise BisectError('Invalid bisect type ' + bisect_type, repo_url)
|
|
|
|
expected_error = _check_for_crash(build_data.project_name, fuzz_target,
|
|
testcase_path)
|
|
logging.info('new_commit result = %s', expected_error)
|
|
|
|
if not should_crash and expected_error:
|
|
logging.warning('new_commit crashed but not shouldn\'t. '
|
|
'Continuing to see if stack changes.')
|
|
|
|
range_valid = False
|
|
for _ in range(2):
|
|
logging.info('Testing against old_commit (%s)', commit_list[old_idx])
|
|
if not build_specified_commit.build_fuzzers_from_commit(
|
|
commit_list[old_idx],
|
|
bisect_repo_manager,
|
|
host_src_dir,
|
|
build_data,
|
|
base_builder_repo=base_builder_repo):
|
|
raise BisectError('Failed to build old_commit', repo_url)
|
|
|
|
if _check_for_crash(build_data.project_name, fuzz_target,
|
|
testcase_path) == expected_error:
|
|
logging.warning('old_commit %s had same result as new_commit %s',
|
|
old_commit, new_commit)
|
|
# Try again on an slightly older commit.
|
|
old_commit = bisect_repo_manager.get_parent(old_commit, 64)
|
|
if not old_commit:
|
|
break
|
|
|
|
commit_list = bisect_repo_manager.get_commit_list(
|
|
new_commit, old_commit)
|
|
old_idx = len(commit_list) - 1
|
|
continue
|
|
|
|
range_valid = True
|
|
break
|
|
|
|
if not range_valid:
|
|
raise BisectError('old_commit had same result as new_commit', repo_url)
|
|
|
|
while old_idx - new_idx > 1:
|
|
curr_idx = (old_idx + new_idx) // 2
|
|
logging.info('Testing against %s (idx=%d)', commit_list[curr_idx],
|
|
curr_idx)
|
|
if not build_specified_commit.build_fuzzers_from_commit(
|
|
commit_list[curr_idx],
|
|
bisect_repo_manager,
|
|
host_src_dir,
|
|
build_data,
|
|
base_builder_repo=base_builder_repo):
|
|
# Treat build failures as if we couldn't repo.
|
|
# TODO(ochang): retry nearby commits?
|
|
old_idx = curr_idx
|
|
continue
|
|
|
|
current_error = _check_for_crash(build_data.project_name, fuzz_target,
|
|
testcase_path)
|
|
logging.info('Current result = %s', current_error)
|
|
if expected_error == current_error:
|
|
new_idx = curr_idx
|
|
else:
|
|
old_idx = curr_idx
|
|
return Result(repo_url, commit_list[new_idx])
|
|
|
|
|
|
# pylint: disable=too-many-locals
|
|
# pylint: disable=too-many-arguments
|
|
def bisect(bisect_type, old_commit, new_commit, testcase_path, fuzz_target,
|
|
build_data):
|
|
"""From a commit range, this function caluclates which introduced a
|
|
specific error from a fuzz testcase_path.
|
|
|
|
Args:
|
|
bisect_type: The type of the bisect ('regressed' or 'fixed').
|
|
old_commit: The oldest commit in the error regression range.
|
|
new_commit: The newest commit in the error regression range.
|
|
testcase_path: The file path of the test case that triggers the error
|
|
fuzz_target: The name of the fuzzer to be tested.
|
|
build_data: a class holding all of the input parameters for bisection.
|
|
|
|
Returns:
|
|
The commit SHA that introduced the error or None.
|
|
|
|
Raises:
|
|
ValueError: when a repo url can't be determine from the project.
|
|
"""
|
|
try:
|
|
return _bisect(bisect_type, old_commit, new_commit, testcase_path,
|
|
fuzz_target, build_data)
|
|
finally:
|
|
# Clean up projects/ as _bisect may have modified it.
|
|
oss_fuzz_repo_manager = repo_manager.RepoManager(helper.OSS_FUZZ_DIR)
|
|
oss_fuzz_repo_manager.git(['reset', 'projects'])
|
|
oss_fuzz_repo_manager.git(['checkout', 'projects'])
|
|
oss_fuzz_repo_manager.git(['clean', '-fxd', 'projects'])
|
|
|
|
|
|
if __name__ == '__main__':
|
|
main()
|