]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
quota: Remove "Failed to set quota transaction limits" error
authorMartti Rannanjärvi <martti.rannanjarvi@dovecot.fi>
Wed, 10 Jan 2018 13:45:51 +0000 (15:45 +0200)
committerAki Tuomi <aki.tuomi@open-xchange.com>
Thu, 1 Feb 2018 10:25:46 +0000 (12:25 +0200)
This error adds nothing that helps the user to debug a problem.

src/plugins/quota/quota-storage.c
src/plugins/quota/quota.c

index bf4b4c01bc8f969e7be38483de4d543860248532..42c8ca7be05d75d32a765ca38a51d49bf10cedb5 100644 (file)
@@ -291,7 +291,7 @@ quota_copy(struct mail_save_context *ctx, struct mail *mail)
        const char *error;
        if (quota_transaction_set_limits(qt, &error_res, &error) < 0 &&
            error_res != QUOTA_GET_RESULT_BACKGROUND_CALC)
-               i_error("Failed to set quota transaction limits: %s", error);
+               i_error("quota: %s", error);
 
        if (qbox->module_ctx.super.copy(ctx, mail) < 0)
                return -1;
@@ -352,7 +352,7 @@ quota_save_begin(struct mail_save_context *ctx, struct istream *input)
        enum quota_get_result error_res;
        if (quota_transaction_set_limits(qt, &error_res, &error) < 0 &&
            error_res != QUOTA_GET_RESULT_BACKGROUND_CALC)
-               i_error("Failed to set quota transaction limits: %s", error);
+               i_error("quota: %s", error);
 
        return qbox->module_ctx.super.save_begin(ctx, input);
 }
index d6086545f5244fddc80d333da880e32ba9802698..362ac2064972520d658991786368dd47a251b56d 100644 (file)
@@ -1308,11 +1308,8 @@ enum quota_alloc_result quota_try_alloc(struct quota_transaction_context *ctx,
        const char *error;
        enum quota_get_result error_res;
 
-       if (quota_transaction_set_limits(ctx, &error_res, &error) < 0) {
-               *error_r = t_strdup_printf(
-                       "Failed to set quota transaction limits: %s", error);
+       if (quota_transaction_set_limits(ctx, &error_res, error_r) < 0)
                return QUOTA_ALLOC_RESULT_TEMPFAIL;
-       }
 
        if (ctx->no_quota_updates)
                return QUOTA_ALLOC_RESULT_OK;
@@ -1353,13 +1350,9 @@ enum quota_alloc_result quota_test_alloc(struct quota_transaction_context *ctx,
                return QUOTA_ALLOC_RESULT_TEMPFAIL;
        }
 
-       const char *error;
        enum quota_get_result error_res;
-       if (quota_transaction_set_limits(ctx, &error_res, &error) < 0) {
-               *error_r = t_strdup_printf(
-                       "Failed to set quota transaction limits: %s", error);
+       if (quota_transaction_set_limits(ctx, &error_res, error_r) < 0)
                return QUOTA_ALLOC_RESULT_TEMPFAIL;
-       }
 
        uoff_t max_size = ctx->quota->set->max_mail_size;
        if (max_size > 0 && size > max_size) {