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 for static analysis issues #128

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Sapna1-singh
Copy link

Below are the issues fixed:

  • Memory - illegal accesses
  • Uninitialized variables
  • Memory - corruptions
  • Resource leak

Tracked-On: OAM-122692

Below are the issues fixed:
- Memory - illegal accesses
- Uninitialized variables
- Memory - corruptions
- Resource leak

Tracked-On: OAM-122692
Signed-off-by: Sapna <[email protected]>
@feijiang1
Copy link

These coverity issues are related upstream code, then we can directly get waiver for them, don't need fix actually. If we need fix them, it is better directly submit to upstream repo.

@sysopenci sysopenci added the Stale Stale label for inactive open prs label Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale label for inactive open prs Valid commit message
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants