From c7da6175bc654a37510541d6d55b7a0010cfd210 Mon Sep 17 00:00:00 2001 From: Matthew Holt Date: Wed, 30 Oct 2019 12:19:59 -0600 Subject: [PATCH] fuzz: Remove admin fuzzer Not really necessary; underlying work is done by json.Unmarshal which is part of the Go standard lib. Also, it called Run, which potentially tries to get certificates; we should not let that happen. --- admin_fuzz.go | 30 ------------------------------ azure-pipelines.yml | 1 - 2 files changed, 31 deletions(-) delete mode 100644 admin_fuzz.go diff --git a/admin_fuzz.go b/admin_fuzz.go deleted file mode 100644 index 6d8095d6..00000000 --- a/admin_fuzz.go +++ /dev/null @@ -1,30 +0,0 @@ -// Copyright 2015 Matthew Holt and The Caddy Authors -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. - -// +build gofuzz -// +build gofuzz_libfuzzer - -package caddy - -import ( - "bytes" -) - -func FuzzAdmin(data []byte) (score int) { - err := Load(bytes.NewReader(data)) - if err != nil { - return 0 - } - return 1 -} diff --git a/azure-pipelines.yml b/azure-pipelines.yml index 34c5db2d..6a56033f 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -215,7 +215,6 @@ jobs: ) declare -A fuzzers_targets=(\ - ["./admin_fuzz.go"]="admin" \ ["./caddyconfig/httpcaddyfile/adapter_fuzz.go"]="caddyfile-adapter" \ ["./caddyconfig/httpcaddyfile/addresses_fuzz.go"]="parse-address" \ ["./caddyconfig/caddyfile/parse_fuzz.go"]="parse-caddyfile" \