diff options
| author |  Austin Clements <amdragon@mit.edu> | 2011-06-10 15:58:18 -0400 |
|---|---|---|
| committer |  David Bremner <bremner@debian.org> | 2011-09-12 23:36:00 -0300 |
| commit | 62445dd0231c4f9e49f0be293e9f539d63bfdc99 (patch) | |
| tree | 9ac8f1459d869dbad591ebba9de50291ebd8edcb | |
| parent | 003e718020b335f329f852ff1a3da23873d2ff6e (diff) | |
lib: Add missing status check in _notmuch_message_remove_filename.
Previously, this function would synchronize the folder list even if
removing the file name failed. Now it returns immediately if removing
the file name fails.
| -rw-r--r-- | lib/message.cc | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lib/message.cc b/lib/message.cc index d993cde8..424081f1 100644 --- a/lib/message.cc +++ b/lib/message.cc @@ -531,6 +531,8 @@ _notmuch_message_remove_filename (notmuch_message_t *message, "file-direntry", direntry); status = COERCE_STATUS (private_status, "Unexpected error from _notmuch_message_remove_term"); + if (status) + return status; /* Re-synchronize "folder:" terms for this message. This requires: * 1. removing all "folder:" terms |