Date: Monday, March 19, 2018 @ 17:03:31 Author: foxxx0 Revision: 309170
upgpkg: dovecot 2.3.0.1-3 add patch for replication issues Added: dovecot/trunk/dovecot-0001-fix-segfault-imap-zlib.patch dovecot/trunk/dovecot-0002-fix-segfault-quota-exceeded.patch dovecot/trunk/dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch Modified: dovecot/trunk/PKGBUILD Deleted: dovecot/trunk/dovecot-fix-segfault-imap-zlib.patch dovecot/trunk/dovecot-fix-segfault-quota-exceeded.patch ---------------------------------------------------------------+ PKGBUILD | 10 dovecot-0001-fix-segfault-imap-zlib.patch | 28 + dovecot-0002-fix-segfault-quota-exceeded.patch | 162 ++++++++++ dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch | 49 +++ dovecot-fix-segfault-imap-zlib.patch | 28 - dovecot-fix-segfault-quota-exceeded.patch | 162 ---------- 6 files changed, 245 insertions(+), 194 deletions(-) Modified: PKGBUILD =================================================================== --- PKGBUILD 2018-03-19 15:15:02 UTC (rev 309169) +++ PKGBUILD 2018-03-19 17:03:31 UTC (rev 309170) @@ -11,7 +11,7 @@ pkgname=dovecot pkgver=2.3.0.1 -pkgrel=2 +pkgrel=3 pkgdesc="An IMAP and POP3 server written with security primarily in mind" url="http://dovecot.org/"; @@ -31,15 +31,17 @@ 'dovecot.sysusersd' 'dovecot.tmpfilesd' # to be checked if upcoming 2.3.1 release has merged them: - 'dovecot-fix-segfault-quota-exceeded.patch' - 'dovecot-fix-segfault-imap-zlib.patch') + 'dovecot-0001-fix-segfault-imap-zlib.patch' + 'dovecot-0002-fix-segfault-quota-exceeded.patch' + 'dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch') sha256sums=('ab772b3e214683aba347203c9391295552255c4d69afb324c7b8c8fc5ad6f153' 'SKIP' 'c5e3a8ffe23e5deb4f7893d9877d972347c2ee45c4ebf713de85c537e47cfcaf' 'bb7620be7fc0217d77fcac940441c2abbd9efc6ef660153288d09ba412e27e06' + 'b8a7b6e324b8b03c02d1e43b700fdb21ca791699bbb046dcc0042f8e6489b2ed' 'e6ef27938a384a54b28c0ce57626dcd738f659465293f2cbb71712cbd5592136' - 'b8a7b6e324b8b03c02d1e43b700fdb21ca791699bbb046dcc0042f8e6489b2ed') + '4946a6cf44d707d2d9c86f4f39c081cbd0373bbfeee7e26fcc9c3033abda7aef') validpgpkeys=( 'E643F0BDFDCD04D9FFCB6279C948525140558AC9' # Timo Sirainen <[email protected]> '2BE74AAB3EE754DFB9C80D3318A348AEED409DA1' # Dovecot Community Edition Added: dovecot-0001-fix-segfault-imap-zlib.patch =================================================================== --- dovecot-0001-fix-segfault-imap-zlib.patch (rev 0) +++ dovecot-0001-fix-segfault-imap-zlib.patch 2018-03-19 17:03:31 UTC (rev 309170) @@ -0,0 +1,28 @@ +From 23da0fa1b30cc11bcc1d467674a0950c527e9ff1 Mon Sep 17 00:00:00 2001 +From: Timo Sirainen <[email protected]> +Date: Sat, 6 Jan 2018 21:22:11 +0200 +Subject: [PATCH] ostream-zlib: Ignore missing finish if parent stream is + ignoring errors + +This fixes panic with imap_zlib plugin when client enables the IMAP COMPRESS +extension and disconnects: + +Panic: file ostream-zlib.c: line 36 (o_stream_zlib_close): assertion failed: (zstream->ostream.finished || zstream->ostream.ostream.stream_errno != 0) +--- + src/lib-compression/ostream-zlib.c | 3 ++- + 1 file changed, 2 insertions(+), 1 deletion(-) + +diff --git a/src/lib-compression/ostream-zlib.c b/src/lib-compression/ostream-zlib.c +index 848ecb7b89..e0b9a91416 100644 +--- a/src/lib-compression/ostream-zlib.c ++++ b/src/lib-compression/ostream-zlib.c +@@ -33,7 +33,8 @@ static void o_stream_zlib_close(struct iostream_private *stream, + struct zlib_ostream *zstream = (struct zlib_ostream *)stream; + + i_assert(zstream->ostream.finished || +- zstream->ostream.ostream.stream_errno != 0); ++ zstream->ostream.ostream.stream_errno != 0 || ++ zstream->ostream.error_handling_disabled); + (void)deflateEnd(&zstream->zs); + if (close_parent) + o_stream_close(zstream->ostream.parent); Added: dovecot-0002-fix-segfault-quota-exceeded.patch =================================================================== --- dovecot-0002-fix-segfault-quota-exceeded.patch (rev 0) +++ dovecot-0002-fix-segfault-quota-exceeded.patch 2018-03-19 17:03:31 UTC (rev 309170) @@ -0,0 +1,162 @@ +From 2bf919786518d138cc07d9cc21e14ad5e07e5e56 Mon Sep 17 00:00:00 2001 +From: Stephan Bosch <[email protected]> +Date: Wed, 17 Jan 2018 21:26:44 +0100 +Subject: [PATCH] lmtp: local: Fix segfault occurring when quota is exceeded. + +--- + src/lmtp/lmtp-local.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c +index fa1ce5d869..5b5fe51a95 100644 +--- a/src/lmtp/lmtp-local.c ++++ b/src/lmtp/lmtp-local.c +@@ -133,7 +133,7 @@ static void + lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, + const char *error) + { +- struct smtp_address *address = rcpt->rcpt.rcpt->path; ++ struct smtp_address *address = rcpt->rcpt.path; + struct lda_settings *lda_set = + mail_storage_service_user_get_set(rcpt->service_user)[2]; + +From cdbcc8db8e0a04b2cbf6ca9f20b3ee7f7173552d Mon Sep 17 00:00:00 2001 +From: Stephan Bosch <[email protected]> +Date: Wed, 31 Jan 2018 10:30:23 +0100 +Subject: [PATCH 1/3] lmtp: local: Make local variable for rcpt->rcpt.rcpt_cmd + in lmtp_local_rcpt_check_quota(). + +--- + src/lmtp/lmtp-local.c | 9 +++++---- + 1 file changed, 5 insertions(+), 4 deletions(-) + +diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c +index c770e35e5b..d0ee4b312e 100644 +--- a/src/lmtp/lmtp-local.c ++++ b/src/lmtp/lmtp-local.c +@@ -195,6 +195,7 @@ static int + lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) + { + struct client *client = rcpt->rcpt.client; ++ struct smtp_server_cmd_ctx *cmd = rcpt->rcpt.rcpt_cmd; + struct smtp_address *address = rcpt->rcpt.path; + struct mail_user *user; + struct mail_namespace *ns; +@@ -245,10 +246,10 @@ lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) + } + + if (ret < 0 && +- !smtp_server_command_is_replied(rcpt->rcpt.rcpt_cmd->cmd)) { +- smtp_server_reply(rcpt->rcpt.rcpt_cmd, +- 451, "4.3.0", "<%s> Temporary internal error", +- smtp_address_encode(address)); ++ !smtp_server_command_is_replied(cmd->cmd)) { ++ smtp_server_reply(cmd, 451, "4.3.0", ++ "<%s> Temporary internal error", ++ smtp_address_encode(address)); + } + return ret; + } + +From c23717da4af9d3275cb45cbc67faaa8daa353ec1 Mon Sep 17 00:00:00 2001 +From: Stephan Bosch <[email protected]> +Date: Wed, 31 Jan 2018 10:34:11 +0100 +Subject: [PATCH 2/3] lmtp: local: Add explicit cmd parameter to + lmtp_local_rcpt_reply_overquota(). + +Using the RCPT cmd is only valid for the RCPT command and not when quota excess +is detected during DATA. That would cause a segmentation fault, since +rcpt->rcpt.rcpt_cmd == NULL. +--- + src/lmtp/lmtp-local.c | 15 +++++++-------- + 1 file changed, 7 insertions(+), 8 deletions(-) + +diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c +index d0ee4b312e..c19c449e61 100644 +--- a/src/lmtp/lmtp-local.c ++++ b/src/lmtp/lmtp-local.c +@@ -134,6 +134,7 @@ lmtp_local_rcpt_deinit(struct lmtp_local_recipient *rcpt) + + static void + lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, ++ struct smtp_server_cmd_ctx *cmd, + const char *error) + { + struct smtp_address *address = rcpt->rcpt.path; +@@ -141,13 +142,11 @@ lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, + mail_storage_service_user_get_set(rcpt->service_user)[2]; + + if (lda_set->quota_full_tempfail) { +- smtp_server_reply(rcpt->rcpt.rcpt_cmd, +- 452, "4.2.2", "<%s> %s", +- smtp_address_encode(address), error); ++ smtp_server_reply(cmd, 452, "4.2.2", "<%s> %s", ++ smtp_address_encode(address), error); + } else { +- smtp_server_reply(rcpt->rcpt.rcpt_cmd, +- 552, "5.2.2", "<%s> %s", +- smtp_address_encode(address), error); ++ smtp_server_reply(cmd, 552, "5.2.2", "<%s> %s", ++ smtp_address_encode(address), error); + } + } + +@@ -232,7 +231,7 @@ lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) + if (ret < 0) { + error = mailbox_get_last_error(box, &mail_error); + if (mail_error == MAIL_ERROR_NOQUOTA) { +- lmtp_local_rcpt_reply_overquota(rcpt, error); ++ lmtp_local_rcpt_reply_overquota(rcpt, cmd, error); + } else { + i_error("mailbox_get_status(%s, STATUS_CHECK_OVER_QUOTA) " + "failed: %s", +@@ -623,7 +622,7 @@ lmtp_local_deliver(struct lmtp_local *local, + } else if (storage != NULL) { + error = mail_storage_get_last_error(storage, &mail_error); + if (mail_error == MAIL_ERROR_NOQUOTA) { +- lmtp_local_rcpt_reply_overquota(rcpt, error); ++ lmtp_local_rcpt_reply_overquota(rcpt, cmd, error); + } else { + smtp_server_reply_index(cmd, rcpt_idx, + 451, "4.2.0", "<%s> %s", + +From f8d9e6c977847a411af9986c9be62f74e4b06143 Mon Sep 17 00:00:00 2001 +From: Stephan Bosch <[email protected]> +Date: Wed, 31 Jan 2018 10:27:54 +0100 +Subject: [PATCH 3/3] lmtp: local: Use recipient index in + lmtp_local_rcpt_reply_overquota(). + +When used during the DATA command, it should send a reply for the correct +recipient. During the RCPT command there is only one reply due. Added assert +that checks this. +--- + src/lmtp/lmtp-local.c | 11 +++++++---- + 1 file changed, 7 insertions(+), 4 deletions(-) + +diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c +index c19c449e61..0b5e7e06ec 100644 +--- a/src/lmtp/lmtp-local.c ++++ b/src/lmtp/lmtp-local.c +@@ -138,15 +138,18 @@ lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, + const char *error) + { + struct smtp_address *address = rcpt->rcpt.path; ++ unsigned int rcpt_idx = rcpt->rcpt.index; + struct lda_settings *lda_set = + mail_storage_service_user_get_set(rcpt->service_user)[2]; + ++ i_assert(rcpt_idx == 0 || rcpt->rcpt.rcpt_cmd == NULL); ++ + if (lda_set->quota_full_tempfail) { +- smtp_server_reply(cmd, 452, "4.2.2", "<%s> %s", +- smtp_address_encode(address), error); ++ smtp_server_reply_index(cmd, rcpt_idx, 452, "4.2.2", "<%s> %s", ++ smtp_address_encode(address), error); + } else { +- smtp_server_reply(cmd, 552, "5.2.2", "<%s> %s", +- smtp_address_encode(address), error); ++ smtp_server_reply_index(cmd, rcpt_idx, 552, "5.2.2", "<%s> %s", ++ smtp_address_encode(address), error); + } + } + Added: dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch =================================================================== --- dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch (rev 0) +++ dovecot-0003-dsync-Revert-to-tmp-if-home-does-not-exist.patch 2018-03-19 17:03:31 UTC (rev 309170) @@ -0,0 +1,49 @@ +From f46b7c33c884b41a614a18187268db351c9d6722 Mon Sep 17 00:00:00 2001 +From: Aki Tuomi <[email protected]> +Date: Mon, 19 Mar 2018 18:39:27 +0200 +Subject: [PATCH] dsync: Revert to /tmp if home does not exist + +Fixes doveadm: Error: Couldn't lock .dovecot-sync.lock: safe_mkstemp(.dovecot-sync.lock) failed: No such file or directory +--- + src/doveadm/dsync/dsync-brain.c | 18 ++++++++++++++++-- + 1 file changed, 16 insertions(+), 2 deletions(-) + +diff --git a/src/doveadm/dsync/dsync-brain.c b/src/doveadm/dsync/dsync-brain.c +index cb4a411..03afdcd 100644 +--- a/src/doveadm/dsync/dsync-brain.c ++++ b/src/doveadm/dsync/dsync-brain.c +@@ -391,6 +391,7 @@ dsync_brain_lock(struct dsync_brain *brain, const char *remote_hostname) + .lock_method = FILE_LOCK_METHOD_FCNTL, + }; + const char *home, *error; ++ struct stat st; + bool created; + int ret; + +@@ -415,8 +416,21 @@ dsync_brain_lock(struct dsync_brain *brain, const char *remote_hostname) + + if (brain->verbose_proctitle) + process_title_set(dsync_brain_get_proctitle_full(brain, DSYNC_BRAIN_TITLE_LOCKING)); +- brain->lock_path = p_strconcat(brain->pool, home, +- "/"DSYNC_LOCK_FILENAME, NULL); ++ ++ /* if homedir does not yet exist, create lock under tmpdir */ ++ if (stat(home, &st) < 0) { ++ if (errno != ENOENT) { ++ i_error("stat(%s) failed: %m", home); ++ return -1; ++ } ++ brain->lock_path = p_strdup_printf(brain->pool, "%s/%s-%s", ++ brain->user->set->mail_temp_dir, ++ brain->user->username, ++ "/"DSYNC_LOCK_FILENAME); ++ } else { ++ brain->lock_path = p_strconcat(brain->pool, home, ++ "/"DSYNC_LOCK_FILENAME, NULL); ++ } + brain->lock_fd = file_create_locked(brain->lock_path, &lock_set, + &brain->lock, &created, &error); + if (brain->lock_fd == -1) +-- +2.1.4 + Deleted: dovecot-fix-segfault-imap-zlib.patch =================================================================== --- dovecot-fix-segfault-imap-zlib.patch 2018-03-19 15:15:02 UTC (rev 309169) +++ dovecot-fix-segfault-imap-zlib.patch 2018-03-19 17:03:31 UTC (rev 309170) @@ -1,28 +0,0 @@ -From 23da0fa1b30cc11bcc1d467674a0950c527e9ff1 Mon Sep 17 00:00:00 2001 -From: Timo Sirainen <[email protected]> -Date: Sat, 6 Jan 2018 21:22:11 +0200 -Subject: [PATCH] ostream-zlib: Ignore missing finish if parent stream is - ignoring errors - -This fixes panic with imap_zlib plugin when client enables the IMAP COMPRESS -extension and disconnects: - -Panic: file ostream-zlib.c: line 36 (o_stream_zlib_close): assertion failed: (zstream->ostream.finished || zstream->ostream.ostream.stream_errno != 0) ---- - src/lib-compression/ostream-zlib.c | 3 ++- - 1 file changed, 2 insertions(+), 1 deletion(-) - -diff --git a/src/lib-compression/ostream-zlib.c b/src/lib-compression/ostream-zlib.c -index 848ecb7b89..e0b9a91416 100644 ---- a/src/lib-compression/ostream-zlib.c -+++ b/src/lib-compression/ostream-zlib.c -@@ -33,7 +33,8 @@ static void o_stream_zlib_close(struct iostream_private *stream, - struct zlib_ostream *zstream = (struct zlib_ostream *)stream; - - i_assert(zstream->ostream.finished || -- zstream->ostream.ostream.stream_errno != 0); -+ zstream->ostream.ostream.stream_errno != 0 || -+ zstream->ostream.error_handling_disabled); - (void)deflateEnd(&zstream->zs); - if (close_parent) - o_stream_close(zstream->ostream.parent); Deleted: dovecot-fix-segfault-quota-exceeded.patch =================================================================== --- dovecot-fix-segfault-quota-exceeded.patch 2018-03-19 15:15:02 UTC (rev 309169) +++ dovecot-fix-segfault-quota-exceeded.patch 2018-03-19 17:03:31 UTC (rev 309170) @@ -1,162 +0,0 @@ -From 2bf919786518d138cc07d9cc21e14ad5e07e5e56 Mon Sep 17 00:00:00 2001 -From: Stephan Bosch <[email protected]> -Date: Wed, 17 Jan 2018 21:26:44 +0100 -Subject: [PATCH] lmtp: local: Fix segfault occurring when quota is exceeded. - ---- - src/lmtp/lmtp-local.c | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c -index fa1ce5d869..5b5fe51a95 100644 ---- a/src/lmtp/lmtp-local.c -+++ b/src/lmtp/lmtp-local.c -@@ -133,7 +133,7 @@ static void - lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, - const char *error) - { -- struct smtp_address *address = rcpt->rcpt.rcpt->path; -+ struct smtp_address *address = rcpt->rcpt.path; - struct lda_settings *lda_set = - mail_storage_service_user_get_set(rcpt->service_user)[2]; - -From cdbcc8db8e0a04b2cbf6ca9f20b3ee7f7173552d Mon Sep 17 00:00:00 2001 -From: Stephan Bosch <[email protected]> -Date: Wed, 31 Jan 2018 10:30:23 +0100 -Subject: [PATCH 1/3] lmtp: local: Make local variable for rcpt->rcpt.rcpt_cmd - in lmtp_local_rcpt_check_quota(). - ---- - src/lmtp/lmtp-local.c | 9 +++++---- - 1 file changed, 5 insertions(+), 4 deletions(-) - -diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c -index c770e35e5b..d0ee4b312e 100644 ---- a/src/lmtp/lmtp-local.c -+++ b/src/lmtp/lmtp-local.c -@@ -195,6 +195,7 @@ static int - lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) - { - struct client *client = rcpt->rcpt.client; -+ struct smtp_server_cmd_ctx *cmd = rcpt->rcpt.rcpt_cmd; - struct smtp_address *address = rcpt->rcpt.path; - struct mail_user *user; - struct mail_namespace *ns; -@@ -245,10 +246,10 @@ lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) - } - - if (ret < 0 && -- !smtp_server_command_is_replied(rcpt->rcpt.rcpt_cmd->cmd)) { -- smtp_server_reply(rcpt->rcpt.rcpt_cmd, -- 451, "4.3.0", "<%s> Temporary internal error", -- smtp_address_encode(address)); -+ !smtp_server_command_is_replied(cmd->cmd)) { -+ smtp_server_reply(cmd, 451, "4.3.0", -+ "<%s> Temporary internal error", -+ smtp_address_encode(address)); - } - return ret; - } - -From c23717da4af9d3275cb45cbc67faaa8daa353ec1 Mon Sep 17 00:00:00 2001 -From: Stephan Bosch <[email protected]> -Date: Wed, 31 Jan 2018 10:34:11 +0100 -Subject: [PATCH 2/3] lmtp: local: Add explicit cmd parameter to - lmtp_local_rcpt_reply_overquota(). - -Using the RCPT cmd is only valid for the RCPT command and not when quota excess -is detected during DATA. That would cause a segmentation fault, since -rcpt->rcpt.rcpt_cmd == NULL. ---- - src/lmtp/lmtp-local.c | 15 +++++++-------- - 1 file changed, 7 insertions(+), 8 deletions(-) - -diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c -index d0ee4b312e..c19c449e61 100644 ---- a/src/lmtp/lmtp-local.c -+++ b/src/lmtp/lmtp-local.c -@@ -134,6 +134,7 @@ lmtp_local_rcpt_deinit(struct lmtp_local_recipient *rcpt) - - static void - lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, -+ struct smtp_server_cmd_ctx *cmd, - const char *error) - { - struct smtp_address *address = rcpt->rcpt.path; -@@ -141,13 +142,11 @@ lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, - mail_storage_service_user_get_set(rcpt->service_user)[2]; - - if (lda_set->quota_full_tempfail) { -- smtp_server_reply(rcpt->rcpt.rcpt_cmd, -- 452, "4.2.2", "<%s> %s", -- smtp_address_encode(address), error); -+ smtp_server_reply(cmd, 452, "4.2.2", "<%s> %s", -+ smtp_address_encode(address), error); - } else { -- smtp_server_reply(rcpt->rcpt.rcpt_cmd, -- 552, "5.2.2", "<%s> %s", -- smtp_address_encode(address), error); -+ smtp_server_reply(cmd, 552, "5.2.2", "<%s> %s", -+ smtp_address_encode(address), error); - } - } - -@@ -232,7 +231,7 @@ lmtp_local_rcpt_check_quota(struct lmtp_local_recipient *rcpt) - if (ret < 0) { - error = mailbox_get_last_error(box, &mail_error); - if (mail_error == MAIL_ERROR_NOQUOTA) { -- lmtp_local_rcpt_reply_overquota(rcpt, error); -+ lmtp_local_rcpt_reply_overquota(rcpt, cmd, error); - } else { - i_error("mailbox_get_status(%s, STATUS_CHECK_OVER_QUOTA) " - "failed: %s", -@@ -623,7 +622,7 @@ lmtp_local_deliver(struct lmtp_local *local, - } else if (storage != NULL) { - error = mail_storage_get_last_error(storage, &mail_error); - if (mail_error == MAIL_ERROR_NOQUOTA) { -- lmtp_local_rcpt_reply_overquota(rcpt, error); -+ lmtp_local_rcpt_reply_overquota(rcpt, cmd, error); - } else { - smtp_server_reply_index(cmd, rcpt_idx, - 451, "4.2.0", "<%s> %s", - -From f8d9e6c977847a411af9986c9be62f74e4b06143 Mon Sep 17 00:00:00 2001 -From: Stephan Bosch <[email protected]> -Date: Wed, 31 Jan 2018 10:27:54 +0100 -Subject: [PATCH 3/3] lmtp: local: Use recipient index in - lmtp_local_rcpt_reply_overquota(). - -When used during the DATA command, it should send a reply for the correct -recipient. During the RCPT command there is only one reply due. Added assert -that checks this. ---- - src/lmtp/lmtp-local.c | 11 +++++++---- - 1 file changed, 7 insertions(+), 4 deletions(-) - -diff --git a/src/lmtp/lmtp-local.c b/src/lmtp/lmtp-local.c -index c19c449e61..0b5e7e06ec 100644 ---- a/src/lmtp/lmtp-local.c -+++ b/src/lmtp/lmtp-local.c -@@ -138,15 +138,18 @@ lmtp_local_rcpt_reply_overquota(struct lmtp_local_recipient *rcpt, - const char *error) - { - struct smtp_address *address = rcpt->rcpt.path; -+ unsigned int rcpt_idx = rcpt->rcpt.index; - struct lda_settings *lda_set = - mail_storage_service_user_get_set(rcpt->service_user)[2]; - -+ i_assert(rcpt_idx == 0 || rcpt->rcpt.rcpt_cmd == NULL); -+ - if (lda_set->quota_full_tempfail) { -- smtp_server_reply(cmd, 452, "4.2.2", "<%s> %s", -- smtp_address_encode(address), error); -+ smtp_server_reply_index(cmd, rcpt_idx, 452, "4.2.2", "<%s> %s", -+ smtp_address_encode(address), error); - } else { -- smtp_server_reply(cmd, 552, "5.2.2", "<%s> %s", -- smtp_address_encode(address), error); -+ smtp_server_reply_index(cmd, rcpt_idx, 552, "5.2.2", "<%s> %s", -+ smtp_address_encode(address), error); - } - } -
