From: Aki Tuomi <aki.tuomi@dovecot.fi>
Date: Wed, 25 Apr 2018 08:04:08 +0000 (+0300)
Subject: doveadm: mailbox cache decisions - Fatally fail if fields are missing
X-Git-Tag: 2.3.9~1943
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=85500a3914798dd5ba457c6bfb34b3c7f90ad443;p=thirdparty%2Fdovecot%2Fcore.git

doveadm: mailbox cache decisions - Fatally fail if fields are missing
---

diff --git a/src/doveadm/doveadm-mail-mailbox-cache.c b/src/doveadm/doveadm-mail-mailbox-cache.c
index 14085f26c7..12d6454e0e 100644
--- a/src/doveadm/doveadm-mail-mailbox-cache.c
+++ b/src/doveadm/doveadm-mail-mailbox-cache.c
@@ -60,9 +60,7 @@ static void cmd_mailbox_cache_decision_init(struct doveadm_mail_cmd_context *_ct
 
 	if (!ctx->all_fields &&
 	    !doveadm_cmd_param_str(_ctx->cctx, "fieldstr", &fields)) {
-		i_error("Missing fields parameter");
-		_ctx->exit_code = EX_USAGE;
-		return;
+		i_fatal("Missing fields parameter");
 	} else if (!ctx->all_fields) {
 		ctx->fields = t_strsplit_spaces(fields, ", ");
 	}
@@ -226,6 +224,9 @@ static int cmd_mailbox_cache_decision_run(struct doveadm_mail_cmd_context *_ctx,
 	const char *const *boxname;
 	int ret = 0;
 
+	if (_ctx->exit_code != 0)
+		return -1;
+
 	for(boxname = ctx->boxes; ret == 0 && *boxname != NULL; boxname++) {
 		struct mailbox *box;
 		if ((ret = cmd_mailbox_cache_open_box(_ctx, user, *boxname, &box)) < 0)