]> Cypherpunks repositories - gostls13.git/commit
[dev.fuzz] internal/fuzz: notify coordinator for minimization
authorJay Conrod <jayconrod@google.com>
Fri, 21 May 2021 16:44:26 +0000 (12:44 -0400)
committerJay Conrod <jayconrod@google.com>
Wed, 2 Jun 2021 20:55:09 +0000 (20:55 +0000)
commitbc1fbe204443cc2e32391b79effe9c18f295d0c2
tree4e31956b9de7655721ed55613f33519fd6676a2e
parenta3b5084204a69e441fa1cc4162a14d93c3507d54
[dev.fuzz] internal/fuzz: notify coordinator for minimization

When a worker process finds a crasher, it now sends that result
directly to the coordinator without attempting to minimize it
first. The coordinator stops sending new inputs and sends the
unminimized crasher back to a worker (any worker) for minimization.

This prevents wasted work during minimization and will help us
implement -keepfuzzing later on. We may also be able to minimize
interesting inputs with this approach later.

Since panics are recoverable errors (they don't terminate worker
processes), we no longer attempt to minimize non-recoverable errors.
This didn't work too well before: we lost too much state.

Change-Id: Id142c7e91a33f64584170b0d42d22cb1f22a92d7
Reviewed-on: https://go-review.googlesource.com/c/go/+/321835
Trust: Jay Conrod <jayconrod@google.com>
Trust: Katie Hockman <katie@golang.org>
Reviewed-by: Katie Hockman <katie@golang.org>
src/cmd/go/testdata/script/test_fuzz_minimize.txt
src/internal/fuzz/fuzz.go
src/internal/fuzz/minimize.go
src/internal/fuzz/minimize_test.go
src/internal/fuzz/worker.go