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

Undefined behavior / Arithmetic overflow in bits.c #25

Closed
Dutchman101 opened this issue Jun 21, 2024 · 3 comments
Closed

Undefined behavior / Arithmetic overflow in bits.c #25

Dutchman101 opened this issue Jun 21, 2024 · 3 comments

Comments

@Dutchman101
Copy link
Contributor

libspeex\bits.c(278): warning C26453: Arithmetic overflow: Left shift of a negative signed number is undefined behavior (io.4)

@ libspeex/bits.c Line 278

@Dutchman101
Copy link
Contributor Author

Dutchman101 commented Oct 3, 2024

Hey @tmatth & @orbea .. is this a maintained repo? I thought you guys were spotting activity here, to carry and file it over at gitlab. I've made a fix at PR #26, ez merge.

If you see this comment, please also look at #27 (that's not pretty either) and perhaps my PR in speexdsp repo.

@tmatth
Copy link
Member

tmatth commented Oct 3, 2024

Hey @tmatth & @orbea .. is this a maintained repo? I thought you guys were spotting activity here, to carry and file it over at gitlab. I've made a fix at PR #26, ez merge.

If you see this comment, please also look at #27 (that's not pretty either) and perhaps my PR in speexdsp repo.

Yeah so as you said, this is just a mirror of the official upstream at https://gitlab.xiph.org/xiph/speexdsp (but we do port patches etc. over)

That said thanks for pinging I will check these ASAP.

ePirat pushed a commit that referenced this issue Oct 4, 2024
@tmatth
Copy link
Member

tmatth commented Oct 4, 2024

Applied in https://gitlab.xiph.org/xiph/speex/-/commit/aca6801183bad01458140dbbab71a68a02e5a561

@tmatth tmatth closed this as completed Oct 4, 2024
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

No branches or pull requests

2 participants