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: revert '+' syntax optimize && add growth check limit for 'random_recursive_mutation' #44

Merged
merged 3 commits into from
Jan 23, 2024

Conversation

0x7Fancy
Copy link
Contributor

for issue #42, I revert '+' syntax optimize

for issue #43, Grammar-Mutator has to rely on AST to work, in the case of TOKEN conflicts, grammar parsing may fail. at the same time, Grammar-Mutator will try to retain data for nodes that fail to parse. If the depth of the syntax tree is large and grammar parsing errors occur, it may cause random_recursive_mutation to be mutated each time. the data size exceeds 1024 bytes, thus falling into do-while. I added an failed limit for this.

@h1994st h1994st changed the base branch from stable to dev January 23, 2024 13:05
@h1994st h1994st merged commit cdad8f0 into AFLplusplus:dev Jan 23, 2024
2 checks passed
@h1994st
Copy link
Collaborator

h1994st commented Jan 23, 2024

LGTM. Thanks!

A kind reminder: please merge to dev branch next time :)

@vanhauser-thc
Copy link
Member

@h1994st when are you pushing to stable? so I can update in the AFL++ repository :)

@h1994st
Copy link
Collaborator

h1994st commented Jan 23, 2024

@vanhauser-thc I can do it now, as there are unlikely new updates recently

@h1994st
Copy link
Collaborator

h1994st commented Jan 23, 2024

@h1994st when are you pushing to stable? so I can update in the AFL++ repository :)

Done.

@vanhauser-thc
Copy link
Member

thank you!

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.

3 participants