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

Update help / usage message #413

Merged

Conversation

goldstar611
Copy link
Contributor

@goldstar611 goldstar611 commented Sep 1, 2023

Hi @ifratric, I'm very excited to see TinyInst support in WinAFL.

I compiled WinAFL with support for TinyInst and ran into some problems initially. The first debug step I did was to run winafl and check the help menu. I was looking for the -y option but it wasn't there. The next thing I did was look at the version and compare with older builds. Neither of these helped me determine if I actually compiled in TinyInst support so I inspected the binary with IDA to confirm.

I proposing a few changes to help others who may face the same situation:

  • bump WINAFL_VERSION to indicate support for TinyInst
  • Add -y option to usage message

@ifratric ifratric merged commit 38297ef into googleprojectzero:master Sep 4, 2023
@ifratric
Copy link
Collaborator

ifratric commented Sep 4, 2023

Makes sense, thanks!

@goldstar611 goldstar611 deleted the usage_message_updates branch September 4, 2023 14:18
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