Merge pull request #5192 from brong/no-intermediate-cache

sync_support: don't cache intermediate records
cyrusimap · Jan 5, 2025 · f5ae3de · f5ae3de
2 parents d2e2787 + 6615e04
commit f5ae3de
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/imap/sync_support.c b/imap/sync_support.c
@@ -6551,14 +6551,13 @@ int sync_do_update_mailbox(struct sync_client_state *sync_cs,
         sync_send_apply(kl, sync_cs->backend->out);
         r = sync_parse_response("MAILBOX", sync_cs->backend->in, NULL);
 
-        // on error, clear cache - otherwise cache this state
-        if (r) sync_uncache(sync_cs, mbentry->name);
-        else r = sync_cache(sync_cs, mbentry->name, kl);
-
         dlist_free(&kl);
         mboxlist_entry_free(&mbentry);
 
-        return 0;
+        // we never want to cache intermediate records
+        sync_uncache(sync_cs, mbentry->name);
+
+        return r;
     }
 
     mboxlist_entry_free(&mbentry);