dovecot-2.2: dsync: Use i_fatal() for invalid parameters, like t...

dovecot at dovecot.org dovecot at dovecot.org
Wed Mar 27 12:53:42 EET 2013


details:   http://hg.dovecot.org/dovecot-2.2/rev/b4040180f30c
changeset: 16123:b4040180f30c
user:      Timo Sirainen <tss at iki.fi>
date:      Wed Mar 27 12:53:35 2013 +0200
description:
dsync: Use i_fatal() for invalid parameters, like the rest of doveadm commands.
Probably the parse_arg() API should be changed to allow reporting errors.

diffstat:

 src/doveadm/dsync/doveadm-dsync.c |  4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diffs (18 lines):

diff -r 37711fb8369f -r b4040180f30c src/doveadm/dsync/doveadm-dsync.c
--- a/src/doveadm/dsync/doveadm-dsync.c	Wed Mar 27 09:16:56 2013 +0200
+++ b/src/doveadm/dsync/doveadm-dsync.c	Wed Mar 27 12:53:35 2013 +0200
@@ -787,12 +787,12 @@
 			ctx->no_mail_sync = TRUE;
 		else if (guid_128_from_string(optarg, ctx->mailbox_guid) < 0 ||
 			 guid_128_is_empty(ctx->mailbox_guid))
-			i_error("Invalid -g parameter: %s", optarg);
+			i_fatal("Invalid -g parameter: %s", optarg);
 		break;
 	case 'l':
 		ctx->lock = TRUE;
 		if (str_to_uint(optarg, &ctx->lock_timeout) < 0)
-			i_error("Invalid -l parameter: %s", optarg);
+			i_fatal("Invalid -l parameter: %s", optarg);
 		break;
 	case 'm':
 		if (optarg[0] == '\0')


More information about the dovecot-cvs mailing list