From e4a484aabe8ca9f3b654612edfdd6114e7235692 Mon Sep 17 00:00:00 2001 From: htuch Date: Tue, 30 Jan 2018 14:35:32 -0500 Subject: [PATCH] coverage: docs and infra/helper.py fix. (#1126) While trying to get a coverage report for a new project (Envoy), building with the coverage sanitizer seemed necessary to get the coverage files dumped. Signed-off-by: Harvey Tuch --- docs/new_project_guide.md | 1 + infra/helper.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/new_project_guide.md b/docs/new_project_guide.md index 09977fdc5..bc69b10e6 100644 --- a/docs/new_project_guide.md +++ b/docs/new_project_guide.md @@ -182,6 +182,7 @@ It's recommended to look at code coverage as a sanity check to make sure that [fuzz target](glossary.md#fuzz-target) gets to the code you expect. ```bash +$ python infra/helper.py build_fuzzers --sanitizer coverage $PROJECT_NAME $ python infra/helper.py coverage $PROJECT_NAME ``` diff --git a/infra/helper.py b/infra/helper.py index ec03157f5..5bb125ba9 100755 --- a/infra/helper.py +++ b/infra/helper.py @@ -183,7 +183,7 @@ def _add_engine_args(parser): def _add_sanitizer_args(parser): """Add common sanitizer args.""" parser.add_argument('--sanitizer', default='address', - choices=['address', 'memory', 'undefined']) + choices=['address', 'memory', 'undefined', 'coverage']) def _add_environment_args(parser):