We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
go version devel go1.22-8eaa7935db Mon Jan 8 18:52:56 2024 +0000 darwin/amd64
go env
irrelevant
mulRange(math.MaxUint64 - 3, math.MaxUint64) overflows the internal median computation (reported by [email protected])
incorrect result / stack overflow
correct result
The text was updated successfully, but these errors were encountered:
Change https://go.dev/cl/554617 mentions this issue: math/big: fix uint64 overflow in nat.mulRange
math/big: fix uint64 overflow in nat.mulRange
Sorry, something went wrong.
aba18d5
e5729ae
Compute median as a + (b-a)/2 instead of (a + b)/2. Add additional test cases. Fixes golang#65025. Change-Id: Ib716a1036c17f8f33f51e33cedab13512eb7e0be Reviewed-on: https://go-review.googlesource.com/c/go/+/554617 Reviewed-by: Robert Griesemer <[email protected]> Auto-Submit: Robert Griesemer <[email protected]> TryBot-Result: Gopher Robot <[email protected]> Run-TryBot: Robert Griesemer <[email protected]> Reviewed-by: Rob Pike <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
griesemer
No branches or pull requests
Go version
go version devel go1.22-8eaa7935db Mon Jan 8 18:52:56 2024 +0000 darwin/amd64
Output of
go env
in your module/workspace:What did you do?
mulRange(math.MaxUint64 - 3, math.MaxUint64) overflows the internal median computation
(reported by [email protected])
What did you see happen?
incorrect result / stack overflow
What did you expect to see?
correct result
The text was updated successfully, but these errors were encountered: