-
Notifications
You must be signed in to change notification settings - Fork 502
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
Incorrect conversion between integer types #5571
Comments
I will work on this issue |
@Nanmozhi22 Thank you for your effort and assistance! |
Hello @Cirilla-zmh Thank you for showing Interest on this issue . Actually I have made a code changes and opened an Internal PR within our Org for review . I will open a public PR shortly. |
As I said in the PR this isn't an issue. The value is compared with a uint8 before conversion so it can't overflow. |
Incorrect conversion of an unsigned 64-bit integer from to a lower bit size type uint8 without an upper bound check.
If a string is parsed into an int using strconv.Atoi, and subsequently that int is converted into another integer type of a smaller size, the result can produce unexpected values.
File Reference : samplers/probability/consistent/tracestate.go:201
The text was updated successfully, but these errors were encountered: