]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
global: Log mailbox_list internal errors
authorMartti Rannanjärvi <martti.rannanjarvi@dovecot.fi>
Fri, 17 Mar 2017 10:58:08 +0000 (12:58 +0200)
committerMartti Rannanjärvi <martti.rannanjarvi@dovecot.fi>
Thu, 23 Mar 2017 09:46:58 +0000 (11:46 +0200)
13 files changed:
src/doveadm/doveadm-mail-index.c
src/doveadm/doveadm-mail-mailbox.c
src/doveadm/doveadm-mail.c
src/doveadm/doveadm-mailbox-list-iter.c
src/doveadm/dsync/dsync-brain-mailbox-tree-sync.c
src/doveadm/dsync/dsync-mailbox-tree-fill.c
src/imap/main.c
src/lib-imap-urlauth/imap-urlauth-backend.c
src/lib-storage/list/mailbox-list-index-backend.c
src/lib-storage/mail-autoexpunge.c
src/plugins/quota/quota-count.c
src/plugins/virtual/virtual-config.c
src/pop3/pop3-client.c

index 0e4ebb3375a6e85b10baacb5eb586dd817353415..2dca85eca033ecff819b6c19afd4ae18636fd5b2 100644 (file)
@@ -206,7 +206,7 @@ cmd_index_run(struct doveadm_mail_cmd_context *_ctx, struct mail_user *user)
        }
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("Listing mailboxes failed: %s",
-                       mailbox_list_get_last_error(user->namespaces->list, NULL));
+                       mailbox_list_get_last_internal_error(user->namespaces->list, NULL));
                doveadm_mail_failed_error(_ctx, MAIL_ERROR_TEMP);
                ret = -1;
        }
index c35e45dc33186c6c38892dbfbf1eb34a93ad90a7..1e96b7f7530af2087edd85280802f745dd596c71 100644 (file)
@@ -741,7 +741,7 @@ cmd_mailbox_path_run(struct doveadm_mail_cmd_context *_ctx,
        if (ret < 0) {
                i_error("Failed to lookup mailbox %s path: %s",
                        ctx->mailbox,
-                       mailbox_list_get_last_error(ns->list, &mail_error));
+                       mailbox_list_get_last_internal_error(ns->list, &mail_error));
                doveadm_mail_failed_error(_ctx, mail_error);
        } else if (ret > 0) {
                doveadm_print(path);
index 4a597b91d363e710e93af8fdb037470772dd0cc0..a609cc7ab8f57f20b4a2a8c6a3134147149d0cf6 100644 (file)
@@ -323,7 +323,7 @@ static int cmd_force_resync_run(struct doveadm_mail_cmd_context *ctx,
        }
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("Listing mailboxes failed: %s",
-                       mailbox_list_get_last_error(user->namespaces->list, NULL));
+                       mailbox_list_get_last_internal_error(user->namespaces->list, NULL));
                doveadm_mail_failed_list(ctx, user->namespaces->list);
                ret = -1;
        }
index 0ddaa87b02d1913505f3cd5b505b22e84ad7597c..d6a99c0f5c6a96ed000110507ac4734bbdd4addf 100644 (file)
@@ -152,7 +152,7 @@ int doveadm_mailbox_list_iter_deinit(struct doveadm_mailbox_list_iter **_iter)
                ret = 0;
        else if ((ret = mailbox_list_iter_deinit(&iter->iter)) < 0) {
                i_error("Listing mailboxes failed: %s",
-                       mailbox_list_get_last_error(iter->user->namespaces->list, &error));
+                       mailbox_list_get_last_internal_error(iter->user->namespaces->list, &error));
                doveadm_mail_failed_error(iter->ctx, error);
        }
        array_free(&iter->patterns);
index bcdad767830c63ea8fd87b02624539fe99b16a33..4ad00fa9130d950d50672794dfe2b641cafe74d9 100644 (file)
@@ -148,7 +148,7 @@ int dsync_brain_mailbox_tree_sync_change(struct dsync_brain *brain,
                if (mailbox_list_delete_dir(change->ns->list, storage_name) == 0)
                        return 0;
 
-               errstr = mailbox_list_get_last_error(change->ns->list, &error);
+               errstr = mailbox_list_get_last_internal_error(change->ns->list, &error);
                if (error == MAIL_ERROR_NOTFOUND ||
                    error == MAIL_ERROR_EXISTS) {
                        dsync_brain_set_changes_during_sync(brain, t_strdup_printf(
index f290f9fd49b0e35f1bf1e28f360694f18ae85906..086b7bbabcca52fa146a634038965bdf09f10e1c 100644 (file)
@@ -361,7 +361,7 @@ int dsync_mailbox_tree_fill(struct dsync_mailbox_tree *tree,
        } T_END;
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("Mailbox listing for namespace '%s' failed: %s",
-                       ns->prefix, mailbox_list_get_last_error(ns->list, error_r));
+                       ns->prefix, mailbox_list_get_last_internal_error(ns->list, error_r));
                ret = -1;
        }
 
@@ -377,7 +377,7 @@ int dsync_mailbox_tree_fill(struct dsync_mailbox_tree *tree,
        }
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("Mailbox listing for namespace '%s' failed: %s",
-                       ns->prefix, mailbox_list_get_last_error(ns->list, error_r));
+                       ns->prefix, mailbox_list_get_last_internal_error(ns->list, error_r));
                ret = -1;
        }
        if (ret < 0)
index 0a4ceb38cff3e66cae68a0dd3873d5d3b6bec01f..583086b72861e8980ba3ed64b6a6846e67107491 100644 (file)
@@ -240,7 +240,7 @@ int client_create_from_input(const struct mail_storage_service_input *input,
           imapc. */
        ns = mail_namespace_find_inbox(mail_user->namespaces);
        (void)mailbox_list_get_hierarchy_sep(ns->list);
-       errstr = mailbox_list_get_last_error(ns->list, &mail_error);
+       errstr = mailbox_list_get_last_internal_error(ns->list, &mail_error);
        if (mail_error != MAIL_ERROR_NONE) {
                *error_r = t_strdup(errstr);
                mail_user_unref(&mail_user);
index 1ea1a31981bfee7c1e4a13a1e2facfedfa2317e2..adee95d13cd43a648fd7c4ccc04d4bdf20b56a6b 100644 (file)
@@ -165,7 +165,7 @@ int imap_urlauth_backend_reset_all_keys(struct mail_user *user)
        }
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("urlauth key reset: Couldn't iterate mailboxes: %s",
-                       mailbox_list_get_last_error(user->namespaces->list, NULL));
+                       mailbox_list_get_last_internal_error(user->namespaces->list, NULL));
                ret = -1;
        }
        return ret;
index c1045e276d15ea3295af6cc7cc63f7a770b3b19a..23bd2f639c94e8ce5f29b00c841982b321d06cb7 100644 (file)
@@ -54,7 +54,7 @@ static int index_list_init(struct mailbox_list *_list, const char **error_r)
        if (mailbox_list_get_root_path(_list, MAILBOX_LIST_PATH_TYPE_INDEX, &dir) &&
            mailbox_list_mkdir_root(_list, dir, MAILBOX_LIST_PATH_TYPE_INDEX) < 0) {
                *error_r = t_strdup_printf("Failed to create the index root directory: %s",
-                                          mailbox_list_get_last_error(_list, NULL));
+                                          mailbox_list_get_last_internal_error(_list, NULL));
                return -1;
        }
        return 0;
index 26dc2c3c4d2fb1e8c2d805e958dd67d8883e73c6..b1f8333db920e916af426f8abfbfddfe1186d507 100644 (file)
@@ -177,7 +177,7 @@ mailbox_autoexpunge_wildcards(struct mail_namespace *ns,
        } T_END;
        if (mailbox_list_iter_deinit(&iter) < 0) {
                i_error("Failed to iterate autoexpunge mailboxes '%s': %s",
-                       iter_name, mailbox_list_get_last_error(ns->list, NULL));
+                       iter_name, mailbox_list_get_last_internal_error(ns->list, NULL));
        }
 }
 
index 5b7019fa030d4d7c7d3bb20af2e01d9c823cc49e..dd1f98206c86988a4ecd9620ba8e78f4a2cc10b9 100644 (file)
@@ -94,7 +94,7 @@ quota_mailbox_iter_deinit(struct quota_mailbox_iter **_iter)
                if (mailbox_list_iter_deinit(&iter->iter) < 0) {
                        i_error("quota: Listing namespace '%s' failed: %s",
                                iter->ns->prefix,
-                               mailbox_list_get_last_error(iter->ns->list, NULL));
+                               mailbox_list_get_last_internal_error(iter->ns->list, NULL));
                        ret = -1;
                }
        }
@@ -130,7 +130,7 @@ quota_mailbox_iter_next(struct quota_mailbox_iter *iter)
        if (mailbox_list_iter_deinit(&iter->iter) < 0) {
                i_error("quota: Listing namespace '%s' failed: %s",
                        iter->ns->prefix,
-                       mailbox_list_get_last_error(iter->ns->list, NULL));
+                       mailbox_list_get_last_internal_error(iter->ns->list, NULL));
                iter->failed = TRUE;
        }
        if (iter->ns->prefix_len > 0 &&
index e8ace07c4542ebf2574756e6df880933d79cb0ca..f56f64f485bf8d302d7383eaf986b914ebcaad96 100644 (file)
@@ -439,7 +439,7 @@ static int virtual_config_expand_wildcards(struct virtual_parse_context *ctx,
        for (i = 0; i < count; i++)
                mail_search_args_unref(&wboxes[i]->search_args);
        if (mailbox_list_iter_deinit(&iter) < 0) {
-               *error_r = mailbox_list_get_last_error(user->namespaces->list, NULL);
+               *error_r = mailbox_list_get_last_internal_error(user->namespaces->list, NULL);
                return -1;
        }
        return ret < 0 ? -1 : 0;
index b02e15ef4881c41d57d2d1f1142b0b9de1c229a2..52f40596139ed51ffec41a77219926c72530c71e 100644 (file)
@@ -353,7 +353,7 @@ static int pop3_lock_session(struct client *client)
        }
        if (mailbox_list_mkdir_root(client->inbox_ns->list, dir, type) < 0) {
                i_error("pop3_lock_session: Couldn't create root directory %s: %s",
-                       dir, mailbox_list_get_last_error(client->inbox_ns->list, NULL));
+                       dir, mailbox_list_get_last_internal_error(client->inbox_ns->list, NULL));
                return -1;
        }
        path = t_strdup_printf("%s/"POP3_LOCK_FNAME, dir);