From: Timo Sirainen Date: Mon, 12 Jun 2017 08:05:31 +0000 (+0300) Subject: imapc: Remove auth_failed check from imapc_list_get_storage_name() X-Git-Tag: 2.3.0.rc1~1437 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a76f1c806e9ac7d8e0d652dfaef5035d0c677892;p=thirdparty%2Fdovecot%2Fcore.git imapc: Remove auth_failed check from imapc_list_get_storage_name() Reverts 9949dc3649ac7a00289f6bf0662c793bee3d4286. It doesn't seem to be actually needed at least now, and it's breaking: * SELECT INBOX -> failure due to remote server being down * SELECT INBOX -> NO [CANNOT] Invalid mailbox name: Name is empty --- diff --git a/src/lib-storage/index/imapc/imapc-list.c b/src/lib-storage/index/imapc/imapc-list.c index 67d600d4f6..1091d95ef7 100644 --- a/src/lib-storage/index/imapc/imapc-list.c +++ b/src/lib-storage/index/imapc/imapc-list.c @@ -347,11 +347,6 @@ imapc_list_get_storage_name(struct mailbox_list *_list, const char *vname) const char *prefix = list->set->imapc_list_prefix; const char *storage_name; - /* check if authentication has failed, if it has, short circuit here - to avoid the error being clobbered by mailbox_list_get_hierarchy_sep */ - if (list->client->auth_failed) - return ""; - storage_name = mailbox_list_default_get_storage_name(_list, vname); if (*prefix != '\0' && strcasecmp(storage_name, "INBOX") != 0) { storage_name = storage_name[0] == '\0' ? prefix :