-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 printf #5556
fuzz printf #5556
Conversation
Co-authored-by: Daniel Hofstetter <daniel.hofstetter@42dh.com>
|
||
fn generate_printf() -> String { | ||
let mut rng = rand::thread_rng(); | ||
let format_specifiers = ["%s", "%d", "%f", "%x", "%o", "%c", "%b", "%q"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about the format specifiers. For example, https://www.gnu.org/savannah-checkouts/gnu/libc/manual/html_node/Table-of-Output-Conversions.html lists many more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, it is just to start :)
GNU testsuite comparison:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks fine to me for a first version, though I'm not sure whether we should merge it yet as @tertsdiepraam works on a rewrite of printf
.
It doesn't matter because it is testing the binaries, it doesn't rely on internal :) |
No description provided.