[squid-dev] [PATCH] Delete cbdata-protected data --with-valgrind-debug

Alex Rousskov rousskov at measurement-factory.com
Wed May 18 06:22:22 UTC 2016


Hello,

    Valgrind was correctly reporting every cbdata allocation in trunk as
leaking! AFAICT, these regressions were introduced by a combination of
trunk r13977 (Bug 4215: double-free in CBDATA) and trunk r13909
(de-duplicate cbdata deallocate actions). These cbdata-protected data
leaks were not present when Squid was built without valgrind support.

I also fixed and polished cbdata debugging that was printing mismatching
Allocating/Freeing pointer values and synced scripts/find-alive.pl.

The surrounding code needs a lot more love than I can currently give it,
but I hope that the attached fixes do not create new problems.


HTH,

Alex.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: free-cbdata-with-valgrind-t2.patch
Type: text/x-diff
Size: 5209 bytes
Desc: not available
URL: <http://lists.squid-cache.org/pipermail/squid-dev/attachments/20160518/f743f830/attachment-0001.patch>


More information about the squid-dev mailing list