Patch for memory leak in vfs_defaults.c
Jeremy Allison
jra at samba.org
Fri Apr 6 23:22:55 UTC 2018
On Sat, Apr 07, 2018 at 12:50:48AM +0200, Timur I. Bakeyev via samba-technical wrote:
>
> Yes, while the fix is trivial by itself, it took a lot of time to locate
> the leak and Andrew did a tremendous work to nail it and fix.
Absolutely ! That's why I want his name on it - as I'm the one
who introduced the bug in the first place, I certainly shouldn't
be the author :-).
> Finding the leak also showed us that there is no good tooling to track
> memleaks in Samba, at least on FreeBSD. Both dmalloc and valgrind failed us
> on various reasons and only jemalloc helped to confirm at that point, that
> the right place was spotted.
Did valgrind not catch this ? That would have seemed the natural
tool to find the problem.
More information about the samba-technical
mailing list