[PATCH] [PUSHED] Possible memory leak in RTFValue

Miklos Vajna vmiklos at suse.cz
Sun Mar 11 13:49:31 PDT 2012


Hi Arnaud,

On Sun, Mar 11, 2012 at 08:56:00PM +0100, Arnaud Versini <arnaud.versini at gmail.com> wrote:
> I'm not sure but Valgrind said that memories allocations in RTFValue are
> not deleted, so this patch make Value's destructor virtual (parent of
> RTFValue).

valgrind is right - the improvement is dramatic! With a testdoc from
fdo#38057, the memory usage after import is 1.8g -> 121m here.

Pushed to master and libreoffice-3-5.

Thanks!

Miklos


More information about the LibreOffice mailing list