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

v1.60.2 reports integer overflow conversion int -> int32 for conversion like int32(group) with group is an int #4935

Closed
6 of 7 tasks
houshengbo opened this issue Aug 21, 2024 · 2 comments
Labels
question Further information is requested

Comments

@houshengbo
Copy link

Welcome

  • Yes, I'm using a binary release within 2 latest releases. Only such installations are supported.
  • Yes, I've searched similar issues on GitHub and didn't find any.
  • Yes, I've read the typecheck section of the FAQ.
  • Yes, I've tried with the standalone linter if available (e.g., gocritic, go vet, etc.).
  • I agree to follow this project's Code of Conduct

Description of the problem

v1.60.2 starts to report "integer overflow conversion int -> int32 for conversion like int32(group) with group is an int", but v1.60.1 does not report this error.

Version of golangci-lint

v1.60.2

Configuration

Using v1.60.2

v1.60.1 is fine.

Go environment

1.22.4

Verbose output of running

G115: integer overflow conversion int -> int32 (gosec)
drqsGroup := int32(group)

group is an int defined.

A minimal reproducible example or link to a public repository

internal project. no link.

Validation

  • Yes, I've included all information above (version, config, etc.).

Supporter

@houshengbo houshengbo added the bug Something isn't working label Aug 21, 2024
Copy link

boring-cyborg bot commented Aug 21, 2024

Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors.

@ldez ldez added question Further information is requested and removed bug Something isn't working labels Aug 21, 2024
@ldez
Copy link
Member

ldez commented Aug 21, 2024

#4927 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants