Hi lamby, > Alas, I don't think I will be able to get to writing a patch/fix today > as I don't fully understand the leak (or, actually, the original > patch!) just yet. > > Lynxis, do you have any quick pointers, however?
original, my patch had the problem, that it use the same stream (compressor context) as the main thread. This seems to be ok, as long the compression is not lzo (I tested with lzma). After your patch to fix lzo, the compressor will be initialized with a new context on every fragment instead only once for all fragments. Also important, all compressor_init's will leak, but since they are only called once, it's not a big problem to leak some kbs. I attached a patch to fix this to the bug #923711 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=923711 It would be nice if you can test it. Best, lynxis PS: If you like to write a small test script, I'm happy to add it to the CI ;)
pgpKO4TiCyQcn.pgp
Description: OpenPGP digital signature
