lib-storage: Never prevent using '/' in mailbox names with mail_full_filesystem_acces...

It was already allowed with LAYOUT=fs, but not with e.g. LAYOUT=Maildir++.
Now it's possible to use e.g. SELECT /home/shareduser/Maildir/test.box

diff --git a/src/lib-storage/mailbox-list.c b/src/lib-storage/mailbox-list.c
index 45086f8a749873a345dc8344910bebc9a9525d06..5ca7c77d1b0473aa562f4178acd4d95a2a1dfc1a 100644 (file)
--- a/src/lib-storage/mailbox-list.c
+++ b/src/lib-storage/mailbox-list.c
@@ -1290,6 +1290,15 @@ mailbox_list_is_valid_fs_name(struct mailbox_list *list, const char *name,
        if (list->mail_set->mail_full_filesystem_access)
                return TRUE;
 
+       /* either the list backend uses '/' as the hierarchy separator or
+          it doesn't use filesystem at all (PROP_NO_ROOT) */
+       if ((list->props & MAILBOX_LIST_PROP_NO_ROOT) == 0 &&
+           mailbox_list_get_hierarchy_sep(list) != '/' &&
+           strchr(name, '/') != NULL) {
+               *error_r = "Name must not have '/' characters";
+               return FALSE;
+       }
+
        /* make sure it's not absolute path */
        if (*name == '/') {
                *error_r = "Begins with '/'";
@@ -1365,15 +1374,6 @@ bool mailbox_list_is_valid_name(struct mailbox_list *list,
                return FALSE;
        }
 
-       /* either the list backend uses '/' as the hierarchy separator or
-          it doesn't use filesystem at all (PROP_NO_ROOT) */
-       if ((list->props & MAILBOX_LIST_PROP_NO_ROOT) == 0 &&
-           mailbox_list_get_hierarchy_sep(list) != '/' &&
-           strchr(name, '/') != NULL) {
-               *error_r = "Name must not have '/' characters";
-               return FALSE;
-       }
-
        return mailbox_list_is_valid_fs_name(list, name, error_r);
 }