Skip to content

Invalid truncation of getopt() result from int to char #181

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.

Sign up for GitHub

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

Jump to bottom
Closed
eyalroz opened this issue Jul 19, 2024 · 0 comments
Closed

Invalid truncation of getopt() result from int to char #181

eyalroz opened this issue Jul 19, 2024 · 0 comments
Labels
bug Something isn't working resolved-on-develop A changeset fixing this issue has been commiutted to the development branch testing Only relevant to the testing programs rather than the main code itself

Comments

@eyalroz
Copy link
Owner

eyalroz commented Jul 19, 2024 •
edited
Loading

Our autotest executable (not part of the library - just testing code) uses the getopt() function.

Adam Bosely said:

getopt returns an int and truncating it to a char causes issues on my system
https://man7.org/linux/man-pages/man3/getopt.3.html

and suggested a fix with PR #177 .
@eyalroz eyalroz added bug Something isn't working resolved-on-develop A changeset fixing this issue has been commiutted to the development branch testing Only relevant to the testing programs rather than the main code itself labels Jul 19, 2024
eyalroz pushed a commit that referenced this issue Jul 19, 2024
@aboseley @eyalroz
Fixes #181 autotest: No longer truncating getopt() result
e2ae9df
eyalroz pushed a commit that referenced this issue Jul 19, 2024
@aboseley @eyalroz
Fixes #181 autotest: No longer truncating getopt() result
29273e9
eyalroz pushed a commit that referenced this issue Jul 25, 2024
@aboseley @eyalroz
Fixes #181 autotest: No longer truncating getopt() result
4ba01a0
phillipjohnston pushed a commit to embeddedartistry/printf that referenced this issue Mar 3, 2025
@aboseley @phillipjohnston
Fixes eyalroz#181 autotest: No longer truncating getopt() result
60229e9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working resolved-on-develop A changeset fixing this issue has been commiutted to the development branch testing Only relevant to the testing programs rather than the main code itself
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eyalroz