Will Bryant wrote:

Hi all/Jonathan,

valgrinding my applications that use zlib reveals that the zlib filter classes leak memory on destruction, as the call to inflateInit2/deflateInit2 is not matched by a corresponding call to inflateEnd/deflateEnd.

I notice that boost/iostreams/filter/zlib.hpp has:

template<typename Alloc> zlib_compressor_impl<Alloc>::~zlib_compressor_impl() { /*reset(true, false);*/ }

and:

template<typename Alloc> zlib_decompressor_impl<Alloc>::~zlib_decompressor_impl() { /*reset(false, false);*/ }

- which I imagine were added in response George M. Garner's note about what I assume was the same problem (http://lists.boost.org/Archives/boost/2005/03/83379.php).

Yes.

Uncommenting those calls to reset fixes the memory leaks. Are they meant to be commented out in the release?

UGGHH! I have no idea why they're commented out. They definitely need to be uncommented. Thanks for pointing it out. Maybe I should post a patch, instead of waiting for 1.33.1.

Cheers, Will

-- Jonathan Turkanis www.kangaroologic.com