Review Request 126105: Fix yet another crash in Dolphin when Baloo isn't running

Boudhayan Gupta bgupta at kde.org
Wed Nov 18 19:40:16 UTC 2015


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

Review request for Baloo, KDE Frameworks, Pinak Ahuja, and Vishesh Handa.


Repository: baloo


Description
-------

Add a check in Baloo::Database::open() to return false if we're opening the database in ReadOnly mode and the database doesn't exist. This fixes a crash in Dolphin when Baloo isn't running.

This isn't the entire fix - one will also have to remove ~/.local/share/baloo/index to not crash anymore; this patch prevents the file from being recreated everytime Baloo::Database::open() is run, and re-causing the crash.


Diffs
-----

  src/engine/database.cpp 4f0579f 

Diff: https://git.reviewboard.kde.org/r/126105/diff/


Testing
-------

Builds, runs, doesn't crash anymore, the works.


Thanks,

Boudhayan Gupta

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151118/07982564/attachment.html>


More information about the Kde-frameworks-devel mailing list