From 29908b9f1375904957e754531912d4ad12e94e74 Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Tue, 5 Jan 2010 15:23:52 -0800 Subject: [PATCH] notmuch new: Avoid updating directory timestamp if interrupted. This was a very dangerous bug. An interrupted "notmuch new" session would still update the timestamp for the directory in the database. This would result in mail files that were not processed due to the original interruption *never* being picked up by future runs of "notmuch new". Yikes! --- notmuch-new.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/notmuch-new.c b/notmuch-new.c index ca68a684..4adbdc7f 100644 --- a/notmuch-new.c +++ b/notmuch-new.c @@ -290,9 +290,11 @@ add_files_recursive (notmuch_database_t *notmuch, next = NULL; } - status = notmuch_directory_set_mtime (directory, path_mtime); - if (status && ret == NOTMUCH_STATUS_SUCCESS) - ret = status; + if (! interrupted) { + status = notmuch_directory_set_mtime (directory, path_mtime); + if (status && ret == NOTMUCH_STATUS_SUCCESS) + ret = status; + } DONE: if (next) -- 2.43.0