Skip to content

Commit

Permalink
cue/scanner: remove go-fuzz testing code and data
Browse files Browse the repository at this point in the history
cue/parser uses cue/scanner, and we already fuzz cue/parser,
so it seems unnecessary to separately fuzz cue/scanner in a way
that doesn't add any coverage.

As a smoke test, I removed one length check in the scanner
to induce an out-of-bounds panic on a specific partial input,
and the cue/parser fuzzer only took three seconds to find the crash.

Signed-off-by: Daniel Martí <mvdan@mvdan.cc>
Change-Id: Ie0aa17a1916b6fb27a30060cd58026064cf682b1
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1194750
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Reviewed-by: Paul Jolly <paul@myitcv.io>
Unity-Result: CUE porcuepine <cue.porcuepine@gmail.com>
  • Loading branch information
mvdan committed May 15, 2024
1 parent c789d3e commit 5472c4b
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 172 deletions.
39 changes: 0 additions & 39 deletions cue/scanner/fuzz.go

This file was deleted.

58 changes: 0 additions & 58 deletions cue/scanner/testdata/corpus/+basic.cue

This file was deleted.

24 changes: 0 additions & 24 deletions cue/scanner/testdata/corpus/+comments.cue

This file was deleted.

1 change: 0 additions & 1 deletion cue/scanner/testdata/corpus/+package.cue

This file was deleted.

49 changes: 0 additions & 49 deletions cue/scanner/testdata/corpus/+tokens.cue

This file was deleted.

1 change: 0 additions & 1 deletion cue/scanner/testdata/cue.mod/module.cue

This file was deleted.

0 comments on commit 5472c4b

Please sign in to comment.