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

Fix coverity issues on branch 8.4 #683

Merged

Conversation

dongbeiouba
Copy link
Member

464364, 464373, 467555 Overflowed constant
456198, 456192, 456200, 456219, 456228, 456230, 456202, 456177, 456168, 456166, 309792, 297704 Logically dead code 456146 Structurally dead code
456221, 456169 Bad bit shift operation
13899, 297759, 456217 Dereference after null check 456213 Array compared against 0
456209 Double unlock
456179, 456201, 456205, 456208, 456151 Unchecked return value 456194 Explicit null dereferenced
456207 Unintentional integer overflow
456132 Data race condition
456116 Argument cannot be negative

Checklist
  • https://yuque.com/tsdoc 增加或更新了必要的文档
  • 增加或更新了必要的测试用例
  • 对于重要修改,更新了CHANGES文件
  • 当前修改存在对已有API参数或返回值的改变
  • 当前修改存在对旧版本功能的兼容性改变(如网络协议或密码算法)

464364, 464373, 467555 Overflowed constant
456198, 456192, 456200, 456219, 456228, 456230, 456202, 456177,
456168, 456166, 309792, 297704 Logically dead code
456146 Structurally dead code
456221, 456169 Bad bit shift operation
13899, 297759, 456217 Dereference after null check
456213 Array compared against 0
456209 Double unlock
456179, 456201, 456205, 456208, 456151 Unchecked return value
456194 Explicit null dereferenced
456207 Unintentional integer overflow
456132 Data race condition
456116 Argument cannot be negative
@dongbeiouba dongbeiouba force-pushed the fix84/coverity_20241223 branch from fd3eea5 to e08c339 Compare December 24, 2024 16:58
@InfoHunter InfoHunter merged commit 04efbb5 into Tongsuo-Project:8.4-stable Dec 27, 2024
90 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants