From 084e60d54a752d9c26263e7540f8687b747203f9 Mon Sep 17 00:00:00 2001 From: David Bremner Date: Sat, 20 Mar 2021 10:10:52 -0300 Subject: [PATCH] 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. --- lib/add-message.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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"); -- 2.43.0