dovecot-2.0: imap: Don't cancel sync if client disconnects in th...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.0/rev/df12f1bc629f
changeset: 9384:df12f1bc629f
user:      Timo Sirainen <tss at iki.fi>
date:      Tue May 26 14:20:23 2009 -0400
description:
imap: Don't cancel sync if client disconnects in the middle.
Fixes "Message count decreased" errors.

diffstat:

1 file changed, 9 insertions(+), 5 deletions(-)
src/imap/imap-sync.c |   14 +++++++++-----

diffs (51 lines):

diff -r 513691dbe21c -r df12f1bc629f src/imap/imap-sync.c
--- a/src/imap/imap-sync.c	Tue May 26 13:48:18 2009 -0400
+++ b/src/imap/imap-sync.c	Tue May 26 14:20:23 2009 -0400
@@ -372,6 +372,10 @@ int imap_sync_more(struct imap_sync_cont
 	string_t *str;
 	int ret = 1;
 
+	/* finish syncing even when client has disconnected. otherwise our
+	   internal state (ctx->messages_count) can get messed up and unless
+	   we immediately stop handling all commands and syncs we could end up
+	   assert-crashing. */
 	str = t_str_new(256);
 	for (;;) {
 		if (ctx->seq == 0) {
@@ -404,7 +408,7 @@ int imap_sync_more(struct imap_sync_cont
 
 			ret = 1;
 			for (; ctx->seq <= ctx->sync_rec.seq2; ctx->seq++) {
-				if (ret <= 0)
+				if (ret == 0)
 					break;
 
 				ret = imap_sync_send_flags(ctx, str);
@@ -426,7 +430,7 @@ int imap_sync_more(struct imap_sync_cont
 				ctx->seq = ctx->sync_rec.seq2;
 			ret = 1;
 			for (; ctx->seq >= ctx->sync_rec.seq1; ctx->seq--) {
-				if (ret <= 0)
+				if (ret == 0)
 					break;
 
 				str_truncate(str, 0);
@@ -452,15 +456,15 @@ int imap_sync_more(struct imap_sync_cont
 
 			ret = 1;
 			for (; ctx->seq <= ctx->sync_rec.seq2; ctx->seq++) {
-				if (ret <= 0)
+				if (ret == 0)
 					break;
 
 				ret = imap_sync_send_modseq(ctx, str);
 			}
 			break;
 		}
-		if (ret <= 0) {
-			/* failure / buffer full */
+		if (ret == 0) {
+			/* buffer full */
 			break;
 		}