quota: Fix namespace deduplication when mailboxes don't have paths.

Nothing requires a mailbox path for quota backends, especially imapc.
Still do namespace deduplication based on alias_for settings.

diff --git a/src/plugins/quota/quota.c b/src/plugins/quota/quota.c
index 3810c132f981a209133c7928492ecc9a7ceb8c06..b8b7b71e56eb33db37536fd10e152ee6547368e0 100644 (file)
--- a/src/plugins/quota/quota.c
+++ b/src/plugins/quota/quota.c
@@ -531,11 +531,19 @@ quota_is_duplicate_namespace(struct quota *quota, struct mail_namespace *ns)
 
        if (!mailbox_list_get_root_path(ns->list,
                                        MAILBOX_LIST_PATH_TYPE_MAILBOX, &path))
-               return TRUE;
+               path = NULL;
 
        namespaces = array_get(&quota->namespaces, &count);
        for (i = 0; i < count; i++) {
-               if (mailbox_list_get_root_path(namespaces[i]->list,
+               /* count namespace aliases only once. don't rely only on
+                  alias_for != NULL, because the alias might have been
+                  explicitly added as the wanted quota namespace. */
+               if (ns->alias_for == namespaces[i] ||
+                   namespaces[i]->alias_for == ns)
+                       continue;
+
+               if (path != NULL &&
+                   mailbox_list_get_root_path(namespaces[i]->list,
                                MAILBOX_LIST_PATH_TYPE_MAILBOX, &path2) &&
                    strcmp(path, path2) == 0) {
                        /* duplicate path */