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

Improve upon splitting by newline when fuzzing with multiple arguments #361

Closed
ericcornelissen opened this issue Jul 31, 2022 · 0 comments
Closed
Labels
fuzz Relates to fuzzing

Comments

@ericcornelissen
Copy link
Owner

The existing implementation for fuzzing with multiple arguments provided to the command being invoked relies on newlines (/[\n\r]+/g) to obtain n arguments (see links below for implementation details). Unfortunately, this means that fuzzing with multiple arguments will never involve any arguments that contain newlines. This limits the scope of the fuzz targets (excluding an important set of arguments, per #332 and related changes). Hence, this is ideally changed such that individual arguments can contain newlines.

Implementation links

@ericcornelissen ericcornelissen added help wanted Extra attention is needed test Relates to testing labels Jul 31, 2022
@ericcornelissen ericcornelissen closed this as not planned Won't fix, can't repro, duplicate, stale Feb 1, 2023
@ericcornelissen ericcornelissen removed the help wanted Extra attention is needed label Feb 1, 2023
@ericcornelissen ericcornelissen added fuzz Relates to fuzzing and removed test Relates to testing labels Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fuzz Relates to fuzzing
Projects
None yet
Development

No branches or pull requests

1 participant