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

[Bug] afl.rs>=0.12.9 passes own arguments to afl-fuzz #273

Open
roypat opened this issue Nov 1, 2022 · 2 comments
Open

[Bug] afl.rs>=0.12.9 passes own arguments to afl-fuzz #273

roypat opened this issue Nov 1, 2022 · 2 comments

Comments

@roypat
Copy link

roypat commented Nov 1, 2022

Starting in version 0.12.9 (coinciding with the upgrade to clap v4.0), arguments passed to cargo afl fuzz are passed down to afl++, where they cause errors as they cannot be parsed. This makes the max_total_time option unusable, as it leads to:

[...]/afl.rs/rustc-1.64.0-a55dd71/afl.rs-0.12.9/afl/bin/afl-fuzz: invalid option -- '-'

[-] PROGRAM ABORT : Bad syntax used for -m
         Location : main(), src/afl-fuzz.c:860

whereas on 0.12.8 everything works as expected

@smoelius
Copy link
Member

smoelius commented Nov 1, 2022

Thank you very much for the bug report. I will get to work on a fix.

But may I please ask, does -V seconds meet your needs?

max_total_time is a bit of an oddball and hence a maintenance burden. If the same functionality can be achieved with -V, then I would like to consider deprecating max_total_time.

@roypat
Copy link
Author

roypat commented Nov 2, 2022

Thank you so much for getting back to me so quickly! Yes, passing -V seconds instead met my needs!

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

No branches or pull requests

2 participants