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 leaks #493

Merged
merged 2 commits into from
Feb 17, 2025
Merged

Fix memory leaks #493

merged 2 commits into from
Feb 17, 2025

Conversation

sni
Copy link
Contributor

@sni sni commented Feb 16, 2025

fix some potential memory leaks. Nothing to worry about but annoying when looking for real memory leaks.

the object ids might change during reloads, so it's better to free the list. Also it does not
pops up as potential memory leak.

Signed-off-by: Sven Nierlein <sven@consol.de>
@sni sni force-pushed the fix_memory_leaks branch from 9c00592 to 50d9fd3 Compare February 16, 2025 23:29
free the previous problem id before overwriting it.
@sni sni force-pushed the fix_memory_leaks branch from 50d9fd3 to c000c54 Compare February 17, 2025 00:16
@sni sni merged commit 08e3302 into naemon:master Feb 17, 2025
36 checks passed
@sni sni deleted the fix_memory_leaks branch February 17, 2025 20:50
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.

2 participants