From: Austin Clements <amdragon@mit.edu>
Date: Fri, 10 Jun 2011 19:58:18 +0000 (-0400)
Subject: lib: Add missing status check in _notmuch_message_remove_filename.
X-Git-Tag: 0.9_rc1~30
X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=commitdiff_plain;h=62445dd0231c4f9e49f0be293e9f539d63bfdc99;hp=003e718020b335f329f852ff1a3da23873d2ff6e

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.
---

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