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 format-truncation on 64bit #1462

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

priv-kweihmann
Copy link

GCC 10.x rightfully complains about out-of-bounds snprintf operation
on 64bit systems, where size_t could be 64bit sized.
Cast bytes_read down to 32bit and printf it without specifying input
format

Fixes #1461

Signed-off-by: Konrad Weihmann [email protected]

GCC 10.x rightfully complains about out-of-bounds snprintf operation
on 64bit systems, where size_t could be 64bit sized.
Cast bytes_read down to 32bit and printf it without specifying input
format

Signed-off-by: Konrad Weihmann <[email protected]>
@priv-kweihmann
Copy link
Author

@BrandonStaab anything I can do to get this merged?

@priv-kweihmann
Copy link
Author

@ALL is there anything I can do to get this here merged?

@Alexander-Degtyar
Copy link

Alexander-Degtyar commented Feb 18, 2021

Merge request helped me. All autotests passed.
gcc 10.2.0
Arch Linux 5.10.16-arch1-1

@Hyask
Copy link

Hyask commented Mar 2, 2021

Any chances to see this merged and released?

@nlogozzo
Copy link

nlogozzo commented Mar 2, 2021

Trying to build on Arch via AUR and getting errors. Building with this PR fixes all issues...please merge and release

@priv-kweihmann
Copy link
Author

Soon this PR will have its first anniversary - is there the slightest chance of getting it at least merged - I mean to me this is a critical bug, as without it even compilation is broken

@BrandonStaab
Copy link

@barcharcraz, can you give us some feedback on this as everyone using a modern compiler gets a compilation error without this PR

@keyonjie
Copy link

keyonjie commented Aug 9, 2022

2 years passed, looks we have to carry this manually with ourselves.

@priv-kweihmann
Copy link
Author

2 years passed, looks we have to carry this manually with ourselves.

Maybe time to drop this PR to increase the pressure 👿 😁

@priv-kweihmann
Copy link
Author

priv-kweihmann commented Aug 22, 2022

@barcharcraz, as the other former maintainers are not responding to the merge this code change here, would you be willing to unblock the compilation for anyone with having gcc 10+?

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.

Can't compile on GCC 10.1
6 participants