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 an possible overflow error #78

Merged
merged 1 commit into from Sep 2, 2018

Conversation

Projects
None yet
2 participants
@ryncsn
Contributor

ryncsn commented Sep 2, 2018

Got:
"specified bound 2048 exceeds the size 19 of the destination"
when -O2 is used, and a "*** buffer overflow detected ***" error output
with no backtrace.

With -O0, it's gone, guess it's some gcc optimization problem, and the
size there is wrong anyway, this patch could fix it.

Fix an possible overflow error
Got:
"specified bound 2048 exceeds the size 19 of the destination"
when -O2 is used, and a "*** buffer overflow detected ***" error output
with no backtrace.

With -O0, it's gone, guess it's some gcc optimization problem, and the
size there is wrong anyway, this patch could fix it.
@ppwaskie

This comment has been minimized.

Show comment
Hide comment
@ppwaskie

ppwaskie Sep 2, 2018

Contributor

This seems reasonable. What version of gcc, just curious?

Contributor

ppwaskie commented Sep 2, 2018

This seems reasonable. What version of gcc, just curious?

@ppwaskie ppwaskie merged commit 5a1c7b8 into Irqbalance:master Sep 2, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ryncsn

This comment has been minimized.

Show comment
Hide comment
@ryncsn

ryncsn Sep 4, 2018

Contributor

gcc --version
gcc (Gentoo Hardened 7.3.0-r3 p1.4) 7.3.0

Thanks for merging

Contributor

ryncsn commented Sep 4, 2018

gcc --version
gcc (Gentoo Hardened 7.3.0-r3 p1.4) 7.3.0

Thanks for merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment