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

fuzz: Add WRITE_ALL_FUZZ_TARGETS_AND_ABORT #21874

Merged

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented May 7, 2021

This is needed when stdout is polluted by the fuzz engine. stderr can't be used instead because it is polluted by aborting the program.

@DrahtBot DrahtBot added the Tests label May 7, 2021
@maflcko maflcko merged commit 9313c4e into bitcoin:master May 7, 2021
@maflcko
Copy link
Member Author

maflcko commented May 7, 2021

Sorry for merging this without review. Post-merge review is welcome.

@maflcko maflcko deleted the 2105-fuzzWRITE_ALL_FUZZ_TARGETS_AND_ABORT branch May 7, 2021 13:50
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 7, 2021
fa5cb6b fuzz: Add WRITE_ALL_FUZZ_TARGETS_AND_ABORT (MarcoFalke)

Pull request description:

  This is needed when stdout is polluted by the fuzz engine. stderr can't be used instead because it is polluted by aborting the program.

Top commit has no ACKs.

Tree-SHA512: bf0a2a6bcd964ff1f0f3ef6e7e297b4c780430c4d6312332ed99ace0e1c58243c1483fd387e39405837d39b36072dfeb9ae03d2a7aa728ad6955159754fd5766
@practicalswift
Copy link
Contributor

practicalswift commented May 7, 2021

Post-merge ACK fa5cb6b: patch looks splendid

Like the C++17 style if (const Foo foo = bar()) {. No more scope creep!

gwillen pushed a commit to ElementsProject/elements that referenced this pull request Jun 1, 2022
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants