[PATCH] [REVIEW] Possible memory leak in RTFValue

Arnaud Versini arnaud.versini at gmail.com
Sun Mar 11 12:56:00 PDT 2012


Hi

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). With this patch, Valgrind doesn't detect leak on this anymore,
but I'm not sure about binary compatibility.

If this is the solution, I will check all others types wich have the same
usage.

Thanks in advance for the review

Best regards

-- 
Arnaud Versini
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120311/dde14ad6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-a-virtual-destrcutor-to-value-to-prevent-memory-leak-in-RTFValue.patch
Type: text/x-patch
Size: 847 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120311/dde14ad6/attachment.bin>


More information about the LibreOffice mailing list