lib: Make notmuch_database_find_message_by_filename not crash on read-only databases

Previously, _notmuch_database_filename_to_direntry would abort with an
internal error when called on a read-only database.  Now that creating
the directory document is optional,
notmuch_database_find_message_by_filename can disable directory
document creation (as it should) and, as a result, not abort on
read-only databases.

diff --git a/lib/database.cc b/lib/database.cc
index e27a0e1664cfac7d2731402f7e3c675c19f57eda..761dc1a24c3a26247dcfa7734ceb7141f02df36a 100644 (file)
--- a/lib/database.cc
+++ b/lib/database.cc
@@ -1895,8 +1895,8 @@ notmuch_database_find_message_by_filename (notmuch_database_t *notmuch,
 
     try {
        status = _notmuch_database_filename_to_direntry (
 
     try {
        status = _notmuch_database_filename_to_direntry (

-           local, notmuch, filename, NOTMUCH_FIND_CREATE, &direntry);
-       if (status)
+           local, notmuch, filename, NOTMUCH_FIND_LOOKUP, &direntry);
+       if (status || !direntry)

            goto DONE;
 
        term = talloc_asprintf (local, "%s%s", prefix, direntry);
            goto DONE;
 
        term = talloc_asprintf (local, "%s%s", prefix, direntry);