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 memory leak reported by clang tool #149

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

Conversation

sarroutbi
Copy link
Collaborator

This memory leak, although not easily reproducible, has been detected by Static Application Security Testing tool. This fix eliminates it.

@sarroutbi sarroutbi changed the title Fix memory leak reported by SAST tool Fix memory leak reported by static analyzer tool May 16, 2024
@sarroutbi sarroutbi force-pushed the 202405161612-fix-possible-memleak branch from f36b3e9 to d531e4b Compare May 16, 2024 14:22
This memory leak, although not easily reproducible,
has been detected by Static Application Security Testing
tool (clang). This fix eliminates it.

Signed-off-by: Sergio Arroutbi <[email protected]>
@sarroutbi sarroutbi force-pushed the 202405161612-fix-possible-memleak branch from d531e4b to c3a5205 Compare May 16, 2024 14:22
@sarroutbi sarroutbi changed the title Fix memory leak reported by static analyzer tool Fix memory leak reported by clang tool May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant