add default path

crytic · Sep 9, 2024 · ba3d9c4 · ba3d9c4
1 parent 37c0f38
commit ba3d9c4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/fuzzing/config/config.go b/fuzzing/config/config.go
@@ -397,9 +397,6 @@ func (p *ProjectConfig) Validate() error {
 
 	// The coverage report format must be either "lcov" or "html"
 	if p.Fuzzing.CoverageReports != nil {
-		if p.Fuzzing.CorpusDirectory == "" {
-			return errors.New("project configuration must specify a corpus directory if coverage reports are enabled")
-		}
 		for _, report := range p.Fuzzing.CoverageReports {
 			if report != "lcov" && report != "html" {
 				return errors.New(fmt.Sprintf("project configuration must specify only valid coverage reports (lcov, html): %s", report))

diff --git a/fuzzing/fuzzer.go b/fuzzing/fuzzer.go
@@ -837,8 +837,12 @@ func (f *Fuzzer) Start() error {
 	f.printExitingResults()
 
 	// Finally, generate our coverage report if we have set a valid corpus directory.
-	if err == nil && f.config.Fuzzing.CorpusDirectory != "" && len(f.config.Fuzzing.CoverageReports) > 0 {
-		coverageReportDir := filepath.Join(f.config.Fuzzing.CorpusDirectory, "coverage")
+	if err == nil && len(f.config.Fuzzing.CoverageReports) > 0 {
+		// Write to the default directory if we have no corpus directory set.
+		coverageReportDir := filepath.Join("crytic-export", "coverage")
+		if f.config.Fuzzing.CorpusDirectory != "" {
+			coverageReportDir = filepath.Join(f.config.Fuzzing.CorpusDirectory, "coverage")
+		}
 		sourceAnalysis, err := coverage.AnalyzeSourceCoverage(f.compilations, f.corpus.CoverageMaps())
 
 		if err != nil {