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 issues found by Coverity Scan in tire.c (branch: 2.6) #15685

Closed
xiaopingcs opened this issue Aug 2, 2022 · 1 comment
Closed

fix issues found by Coverity Scan in tire.c (branch: 2.6) #15685

xiaopingcs opened this issue Aug 2, 2022 · 1 comment
Labels
good first issue Good for newcomers improvement Improve the code readability

Comments

@xiaopingcs
Copy link
Contributor

Bug Description
fix issues found by Coverity Scan in TDengine/src/kit/shell/src/tire.c

To Reproduce
N/A

Expected Behavior
fix following issues:

Type Category Function Line Number
Missing break in switch Control flow issues matchPrefix 334
Use after free Memory - illegal accesses enumAll Various
Missing break in switch Control flow issues enumAll 396
Resource leak Resource leaks matchPrefixFromTree 324
Resource leak Resource leaks enumFromTree 386
Use after free Memory - illegal accesses enumFromTree Various
Copy into fixed size buffer Security best practices violations enumAllWords 249

Screenshots
N/A

Environment (please complete the following information):
N/A

Additional Context
above issues found by https://scan.coverity.com/projects/tdengine?tab=overview

@xiaopingcs xiaopingcs added bug Something isn't working good first issue Good for newcomers improvement Improve the code readability and removed bug Something isn't working labels Aug 2, 2022
@Azanul
Copy link

Azanul commented Jun 5, 2023

Is this issue still relevant? The file seems to have been moved from TDengine/src/kit/shell/src/tire.c

@yu285 yu285 closed this as completed May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers improvement Improve the code readability
Projects
None yet
Development

No branches or pull requests

3 participants