Review Request: Prevent memory leaks when opening a zip store multiple times

Boudewijn Rempt boud at valdyas.org
Wed Oct 6 15:15:56 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5536/
-----------------------------------------------------------

(Updated 2010-10-06 14:15:56.188293)


Review request for kdelibs.


Changes
-------

Added the same code to bzip2 and kxz.


Summary
-------

It is possible to accidentally call open() twice. If you do that on a zip store, kgzipfilter will be initialized twice, deflateInit2 will be called twice and we leak memory.


Diffs (updated)
-----

  trunk/KDE/kdelibs/kdecore/compression/kbzip2filter.cpp 1182756 
  trunk/KDE/kdelibs/kdecore/compression/kfilterdev.cpp 1182756 
  trunk/KDE/kdelibs/kdecore/compression/kgzipfilter.cpp 1182756 
  trunk/KDE/kdelibs/kdecore/compression/kxzfilter.cpp 1182756 

Diff: http://svn.reviewboard.kde.org/r/5536/diff


Testing
-------


Thanks,

Boudewijn

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20101006/4dd665b6/attachment.htm>


More information about the kde-core-devel mailing list