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 8.3 branch #682

Merged

Conversation

dongbeiouba
Copy link
Member

470740, 470741, 470742 Explicit null dereferenced
464373 Overflowed constant
265647 Explicit null dereferenced
464516, 456146 Structurally dead code
297397, 297533, 297551, 456144, 456113, 297761, 297738, 297685, 297667, 297378, 297643, 297577, 456151 Unchecked return value 456130, 456136 Unchecked return value from library 456133 Explicit null dereferenced
456132 Data race condition
456116 Argument cannot be negative
13896, 297759, 297445 Dereference after null check 297704 Logically dead code
13891 Division or modulo by zero

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

@dongbeiouba dongbeiouba requested review from InfoHunter, wa5i and a team December 18, 2024 10:17
@dongbeiouba dongbeiouba self-assigned this Dec 18, 2024
@dongbeiouba dongbeiouba force-pushed the fix83/coverity241211 branch 3 times, most recently from 1d698e2 to 4e952d6 Compare December 22, 2024 12:27
470740, 470741, 470742 Explicit null dereferenced
464373 Overflowed constant
265647 Explicit null dereferenced
464516, 456146 Structurally dead code
297397, 297533, 297551, 456144, 456113, 297761, 297738, 297685, 297667,
297378, 297643, 297577, 456151 Unchecked return value
456130, 456136 Unchecked return value from library
456133 Explicit null dereferenced
456132 Data race condition
456116 Argument cannot be negative
13896, 297759, 297445 Dereference after null check
297704 Logically dead code
13891 Division or modulo by zero
@InfoHunter InfoHunter merged commit 3387b33 into Tongsuo-Project:8.3-stable Dec 23, 2024
71 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