This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117930/

This review has been submitted with commit 59519f5b7f69d3c661379765b4928355cd2e047d by Frank Reininghaus to branch KDE/4.13.

- Commit Hook


On May 1st, 2014, 1:14 p.m. UTC, Frank Reininghaus wrote:

Review request for Baloo.
By Frank Reininghaus.

Updated May 1, 2014, 1:14 p.m.

Bugs: 333763
Repository: baloo

Description

https://bugs.kde.org/show_bug.cgi?id=333763 shows that the Xapian database constructor can throw an exception, which is not handled at the moment. This patch wraps all Xapian-related stuff in Baloo::FileFetchJob::doStart() in a try {...} block and catches all exceptions.

The alternative would be to only add a catch statement for the Xapian::DatabaseOpeningError which seems to be thrown according to the bug report, provided that we can be 100% sure that no other exceptions can be thrown there. IMHO, a library that uses external libraries that can throw exceptions should try as hard as possible to catch all exceptions.

Testing

Unit tests still pass.

Diffs

  • src/file/lib/filefetchjob.cpp (21f5520)

View Diff