diff options
| author | David Bremner <david@tethera.net> | 2021-03-20 10:10:52 -0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2021-04-18 10:02:20 -0300 |
| commit | 084e60d54a752d9c26263e7540f8687b747203f9 (patch) | |
| tree | 9133f8876a5893f726081c4d9efcd10aadcd6de6 /lib | |
| parent | 828a004921ab0736634203cd07cc41393da00047 (diff) | |
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.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/add-message.cc | 4 |
1 files changed, 3 insertions, 1 deletions
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"); |