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.

Sign up for GitHub

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

Jump to bottom

networkd: plug a memory leak #344

Merged

slyon merged 1 commit into canonical:main from daniloegea:fix_leak_write_wpa_conf

Apr 24, 2023

Collaborator

daniloegea commented Apr 20, 2023 •

edited by slyon

Loading

Description

Believe or not, it was found with this https://github.com/daniloegea/netplan-fuzz 👀

Checklist

Runs make check successfully.
Retains 100% code coverage (make check-coverage).
New/changed keys in YAML format are documented.
(Optional) Adds example YAML for new feature.
(Optional) Closes an open bug in Launchpad.


          networkd: plug a memory leak

9dc4c17

daniloegea requested a review from slyon

April 20, 2023 15:17

slyon approved these changes

View reviewed changes

Collaborator

slyon left a comment

+1 nice find!

We should get your "netplan-fuzz" tool (and other fuzzing) integrated into a CI workflow.

slyon merged commit 14de061 into canonical:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet