-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Continuous Fuzzing Integration to Fuzzit #3093
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -44,6 +44,26 @@ ifeq ($(TEST_TYPE),coverage) | |
fi; \ | ||
done | ||
endif | ||
ifeq ($(TEST_TYPE),fuzzit) | ||
# skip fuzzing for PR | ||
if [ "$(TRAVIS_PULL_REQUEST)" = "false" ] || [ "$(FUZZIT_TYPE)" = "local-regression" ] ; then \ | ||
export GO111MODULE=off; \ | ||
go get -u github.com/dvyukov/go-fuzz/go-fuzz-build; \ | ||
go get -u -v .; \ | ||
cd ../../go-acme/lego; \ | ||
git checkout v2.5.0; \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. oh wow, this is pretty horrible :/ I think it make sense to make this into Makefile.fuzz under some phony install target, so we can call There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the workaround for the strange glitch with go-acme v3. How do you suggest working around that? (not sure I got it regarding the make install) |
||
cd ../../coredns/coredns; \ | ||
LIBFUZZER=YES make -f Makefile.fuzz cache chaos file rewrite whoami corefile; \ | ||
wget -O fuzzit https://github.com/fuzzitdev/fuzzit/releases/download/v2.4.27/fuzzit_Linux_x86_64; \ | ||
chmod a+x fuzzit; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/cache ./cache; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/chaos ./chaos; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/file ./file; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/rewrite ./rewrite; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/whoami ./whoami; \ | ||
./fuzzit create job --type $(FUZZIT_TYPE) coredns/corefile ./corefile; \ | ||
fi; | ||
endif | ||
|
||
core/plugin/zplugin.go core/dnsserver/zdirectives.go: plugin.cfg | ||
GO111MODULE=on go generate coredns.go | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comes out as
if [ "3093" = "false" ] || [ "fuzzing" = "local-regression" ] ; then \
is that correct?