Skip to content
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

Fix the setup of Grimoire #2048

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

renatahodovan
Copy link

The previous version of fuzzer.py was possibly copy-pasted from Nautilus and applied unexpected CLI setup. The patch fixes these issues.

@renatahodovan
Copy link
Author

Honestly, I'm not an expert on Grimoire, but I encountered this issue while trying to execute it. As far as I know, Grimoire doesn't require a grammar, and looking into the implementation of its CLI supports this here.

@DonggeLiu @addisoncrump You seem familiar with LibAFL-based fuzzers. Could you please take a look at this?

@renatahodovan
Copy link
Author

@addisoncrump I'm not sure that my patch above is correct. Even if neither -r, nor -g are supported by the Grimoire implementation of libafl_fuzzbench, but while -g is reasonable since Grimoire doesn't use a grammar, but -r might be needed to avoid putting the internal representation of the test cases to the output corpus. But the -r support should be implemented in libafl_fuzzbench. Or am I wrong?

The previous version of fuzzer.py was possibly copy-pasted from
Nautilus and applied unexpected CLI setup. The patch fixes these
issues.
@addisoncrump
Copy link
Contributor

Hey, sorry for the delay on response. The repo you're pulling from is somewhat outdated, and I've not used that fuzzer. Instead, you might consider looking at (what used to be called) libafl_text from some revisions ago. We removed it because we found that grimoire was not relevant for the targets being benchmarked for the most part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants