Willy,
this series fixes up a few more frees. This time I have verified the changes
a bit more carefully, running configuration check on a real world configuration
of mine within valgrind. It still reports a five leaks (but less than without
applying these patches!) and does not report any memory unsafety / bogus frees
/ double frees / use-after-frees.
The first two patches fix leaks within helper functions. I triggered them
during deinit, but they might or might not be accessible during normal
operation (e.g. via the CLI socket). The last two patches are part of the
deinit() function and thus cannot happen in normal operation, but cleaning
them up makes it easier to find actual, important, issues.
Feel free to apply to 'next' only (or after the release of 2.2). No need to
risk breaking anything shortly before release with something less important
like this. And of course please carefully review them :-)
Best regards
Tim Düsterhus (4):
BUG/MINOR: acl: Fix freeing of expr->smp in prune_acl_expr
BUG/MINOR: sample: Fix freeing of conv_exprs in release_sample_expr
BUG/MINOR: haproxy: Free proxy->format_unique_id during deinit
BUG/MINOR: haproxy: Add missing free of
server->(hostname|resolvers_id)
src/acl.c | 4 ++--
src/haproxy.c | 9 +++++++++
src/sample.c | 6 +++++-
3 files changed, 16 insertions(+), 3 deletions(-)
--
2.27.0
