From: David Bremner <david@tethera.net>
Date: Sat, 20 Mar 2021 13:10:52 +0000 (-0300)
Subject: lib/n_d_index_file: check return value from _n_m_add_filename
X-Git-Tag: 0.32_rc0~5
X-Git-Url: https://git.cworth.org/git?p=notmuch;a=commitdiff_plain;h=084e60d54a752d9c26263e7540f8687b747203f9

lib/n_d_index_file: check return value from _n_m_add_filename

Ignoring this return value seems like a bad idea in general, and in
particular it has been hiding one or more bugs related to handling
long directory names.
---

diff --git a/lib/add-message.cc b/lib/add-message.cc
index 485debad..0c34d318 100644
--- a/lib/add-message.cc
+++ b/lib/add-message.cc
@@ -529,7 +529,9 @@ notmuch_database_index_file (notmuch_database_t *notmuch,
 	    goto DONE;
 	}
 
-	_notmuch_message_add_filename (message, filename);
+	ret = _notmuch_message_add_filename (message, filename);
+	if (ret)
+	    goto DONE;
 
 	if (is_new || is_ghost) {
 	    _notmuch_message_add_term (message, "type", "mail");