diff options
| author | Jay Conrod <jayconrod@google.com> | 2021-09-14 15:09:06 -0700 |
|---|---|---|
| committer | Jay Conrod <jayconrod@google.com> | 2021-09-15 20:08:50 +0000 |
| commit | 4304cf62e9ba6a85e37d5fec33cfa580ce7ac6d1 (patch) | |
| tree | 76e392d2037bf47b62b4bbb8a1e57cfec4584f3f | |
| parent | f53b61d38793dfa8987378cbb6754365569c261b (diff) | |
| download | go-4304cf62e9ba6a85e37d5fec33cfa580ce7ac6d1.tar.gz go-4304cf62e9ba6a85e37d5fec33cfa580ce7ac6d1.zip | |
[dev.fuzz] testing: fix internal error exit condition for fuzz workers
A fuzz worker process should exit with fuzzWorkerExitCode (70) if it
encounters an internal error.
This should generally only happen due to an I/O error on the worker
side. I can't think of a way to write a realistic test case for
this. test_fuzz_io_error.txt triggers an I/O error on the coordinator
side, which is similar.
Change-Id: I40d2e9aaf422e5ed925debcf7a152c252c3a6984
Reviewed-on: https://go-review.googlesource.com/c/go/+/349993
Trust: Jay Conrod <jayconrod@google.com>
Trust: Katie Hockman <katie@golang.org>
Reviewed-by: Katie Hockman <katie@golang.org>
| -rw-r--r-- | src/testing/testing.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/testing/testing.go b/src/testing/testing.go index f4d2b26650..18a0657561 100644 --- a/src/testing/testing.go +++ b/src/testing/testing.go @@ -1635,7 +1635,7 @@ func (m *M) Run() (code int) { } fuzzingOk := runFuzzing(m.deps, m.fuzzTargets) - if !*isFuzzWorker && !fuzzingOk { + if !fuzzingOk { fmt.Println("FAIL") if *isFuzzWorker { m.exitCode = fuzzWorkerExitCode |