Review Request 128888: Improve epub extractor, less segfaults
Christoph Cullmann
cullmann at kde.org
Sun Sep 11 12:51:27 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128888/
-----------------------------------------------------------
Review request for KDE Frameworks and Pinak Ahuja.
Repository: kfilemetadata
Description
-------
Improve epub extractor:
1) check for more nullpointers (e.g. data can be null for some fields, iterators, ...)
2) actually close the epub file again at all
3) iterator seems to handle clink as stated in docs, fix double free
e.g. see bug 361727
could be the double freed clink in the last iterator
https://bugs.kde.org/show_bug.cgi?id=361727
Diffs
-----
src/extractors/epubextractor.cpp 88a4caa
Diff: https://git.reviewboard.kde.org/r/128888/diff/
Testing
-------
Unit test still works, no idea how to craft files broken enough to trigger the faults
Thanks,
Christoph Cullmann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160911/5f2b5d54/attachment.html>
More information about the Kde-frameworks-devel
mailing list