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 possible memory leak and NULL dereference #38

Merged
merged 2 commits into from
May 23, 2020

Conversation

wolframroesler
Copy link
Contributor

Avoid potential memory leak when loading the config file, and potential NULL pointer dereference when processing a multicast packet. Both happen under obscure circumstances only (I think). Spotted during static analysis with clang-tidy-9.

This PR fixes the last findings we got from clang-tidy, with the exception of the one described in #37.

Wolfram Rösler added 2 commits May 22, 2020 16:33
Spotted by static analysis with clang-tidy.
Spotted during static analysis with clang-tidy.
@troglobit
Copy link
Owner

Awesome work on this, thanks!

@troglobit troglobit merged commit ef3e351 into troglobit:master May 23, 2020
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