dovecot: If UIDVALIDITY lookup fails, don't return failure to wh...

dovecot at dovecot.org dovecot at dovecot.org
Wed Jul 18 00:03:27 EEST 2007


details:   http://hg.dovecot.org/dovecot/rev/f35ef260987c
changeset: 6047:f35ef260987c
user:      Timo Sirainen <tss at iki.fi>
date:      Wed Jul 18 00:03:14 2007 +0300
description:
If UIDVALIDITY lookup fails, don't return failure to whole APPEND, just
don't return APPENDUID.

diffstat:

1 file changed, 3 insertions(+), 7 deletions(-)
src/imap/cmd-append.c |   10 +++-------

diffs (20 lines):

diff -r b407586a5107 -r f35ef260987c src/imap/cmd-append.c
--- a/src/imap/cmd-append.c	Wed Jul 18 00:02:31 2007 +0300
+++ b/src/imap/cmd-append.c	Wed Jul 18 00:03:14 2007 +0300
@@ -258,13 +258,9 @@ static bool cmd_append_continue_parsing(
 		i_assert(ctx->count == uid2 - uid1 + 1);
 
 		if (mailbox_get_status(ctx->box, STATUS_UIDVALIDITY,
-				       &status) < 0) {
-			client_send_storage_error(cmd, ctx->storage);
-			cmd_append_finish(ctx);
-			return TRUE;
-		}
-
-		if (uid1 == uid2) {
+				       &status) < 0)
+			msg = "OK Append completed. UIDVALIDITY lookup failed.";
+		else if (uid1 == uid2) {
 			msg = t_strdup_printf("OK [APPENDUID %u %u] "
 					      "Append completed.",
 					      status.uidvalidity, uid1);


More information about the dovecot-cvs mailing list