# HG changeset patch
# User Timo Sirainen <timo.sirainen@dovecot.fi>
# Date 1518789826 -7200
# Node ID 61143106dee6201126ad5acc5125382e7920a1d5
# Parent  b47c291ae2af25fca094d2115a444e1970baec22
doveadm sync/backup: Don't override BROKENCHAR if it's already set

This allows migrating invalid mailbox names by specifying BROKENCHAR.
Previously it would always try to use \003 control character, which isn't
valid character in mailbox names so the mailbox creation would fail.

diff -r b47c291ae2af -r 61143106dee6 src/doveadm/doveadm-dsync.c
--- a/src/doveadm/doveadm-dsync.c	Mon Feb 26 18:20:30 2018 +0200
+++ b/src/doveadm/doveadm-dsync.c	Fri Feb 16 16:03:46 2018 +0200
@@ -323,8 +323,10 @@
 	struct mail_namespace *ns;
 
 	user->dsyncing = TRUE;
-	for (ns = user->namespaces; ns != NULL; ns = ns->next)
-		ns->list->set.broken_char = DSYNC_LIST_BROKEN_CHAR;
+	for (ns = user->namespaces; ns != NULL; ns = ns->next) {
+		if (ns->list->set.broken_char == '\0')
+			ns->list->set.broken_char = DSYNC_LIST_BROKEN_CHAR;
+	}
 }
 
 static bool paths_are_equal(struct mail_user *user1, struct mail_user *user2,