imapc: Avoid duplicate FETCH BODY.PEEK[] when mail is already expunged

When the first FETCH (e.g. as part of mail_prefetch()) indicated that the
mail is already expunged, there's no need to do it again.

diff --git a/src/lib-storage/index/imapc/imapc-mail.c b/src/lib-storage/index/imapc/imapc-mail.c
index 27a20fcb97a5df30069499102a3b144d838e3c91..23a32ab3a52bbd481a94b5a4a4178e90f3868e50 100644 (file)
--- a/src/lib-storage/index/imapc/imapc-mail.c
+++ b/src/lib-storage/index/imapc/imapc-mail.c
@@ -312,6 +312,12 @@ imapc_mail_get_stream(struct mail *_mail, bool get_body,
                        mail_set_aborted(_mail);
                        return -1;
                }
+               if (_mail->expunged) {
+                       /* We already detected that the mail is expunged.
+                          Don't spend time trying to FETCH it again. */
+                       mail_set_expunged(_mail);
+                       return -1;
+               }
                fetch_field = get_body ||
                        (data->access_part & READ_BODY) != 0 ?
                        MAIL_FETCH_STREAM_BODY : MAIL_FETCH_STREAM_HEADER;