From 62445dd0231c4f9e49f0be293e9f539d63bfdc99 Mon Sep 17 00:00:00 2001 From: Austin Clements Date: Fri, 10 Jun 2011 15:58:18 -0400 Subject: [PATCH] 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. --- lib/message.cc | 2 ++ 1 file changed, 2 insertions(+) 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