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

[ioctl] incorrect conversion between integer types #3524

Closed
1 task done
dustinxie opened this issue Jul 12, 2022 · 0 comments · Fixed by #3522
Closed
1 task done

[ioctl] incorrect conversion between integer types #3524

dustinxie opened this issue Jul 12, 2022 · 0 comments · Fixed by #3522
Assignees
Labels

Comments

@dustinxie
Copy link
Member

dustinxie commented Jul 12, 2022

Description of the code refactor

fix all "Incorrect conversion between integer types" in the security scanning result

don't fix #11 https://github.com/iotexproject/iotex-core/security/code-scanning/11, it will be handled separately

Why is this needed

  • [] better/less memory usage
  • [] reduce running time
  • [] clean (easy-to-understand) data structure or algorithm
  • [] remove dead code
  • better code design or structure (potential security issue)
  • [] reduce/decouple dependencies between code modules
  • [] increase test coverage
  • [] better code readability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants