dovecot-2.1: dsync: Added workaround to fail if there are UTF8 m...

[dovecot at dovecot.org]

details:   http://hg.dovecot.org/dovecot-2.1/rev/a88aca17a92c
changeset: 14977:a88aca17a92c
user:      Timo Sirainen <tss at iki.fi>
date:      Wed Jun 05 18:25:32 2013 +0300
description:
dsync: Added workaround to fail if there are UTF8 mailbox names in filesystem.

diffstat:

 src/doveadm/dsync/dsync-worker-local.c |  8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diffs (18 lines):

diff -r 7d34543189c6 -r a88aca17a92c src/doveadm/dsync/dsync-worker-local.c
--- a/src/doveadm/dsync/dsync-worker-local.c	Sat Jun 01 02:56:57 2013 +0300
+++ b/src/doveadm/dsync/dsync-worker-local.c	Wed Jun 05 18:25:32 2013 +0300
@@ -556,7 +556,13 @@
 		i_error("Failed to sync mailbox %s: %s", info->name,
 			mailbox_get_last_error(box, &error));
 		mailbox_free(&box);
-		if (error == MAIL_ERROR_NOTFOUND ||
+		/* don't check for NOTFOUND in here anymore. if it happens
+		   because of a race condition it's pretty bad luck. more
+		   importantly it happens if there's a UTF8 mailbox name in
+		   filesystem while it should have been mUTF7 and we can't
+		   easily detect this in another way. this has been fixed
+		   properly in v2.2. */
+		if (/*error == MAIL_ERROR_NOTFOUND ||*/
 		    error == MAIL_ERROR_NOTPOSSIBLE) {
 			/* Mailbox isn't selectable, try the next one. We
 			   should have already caught \Noselect mailboxes, but