https://bugs.exim.org/show_bug.cgi?id=2525
Git Commit <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #1 from Git Commit <[email protected]> --- Git commit: https://git.exim.org/exim.git/commitdiff/5bf8a51681e171328e72f5d5b5ef8fd8a67d5f05 commit 5bf8a51681e171328e72f5d5b5ef8fd8a67d5f05 Author: Jeremy Harris <[email protected]> AuthorDate: Sun Feb 16 21:03:46 2020 +0000 Commit: Jeremy Harris <[email protected]> CommitDate: Mon Feb 17 13:53:18 2020 +0000 Fix continued-connection SIZE parameter on MAIL commmand. Bug 2525 --- doc/doc-txt/ChangeLog | 5 +++++ src/src/transports/smtp.c | 3 +-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index 22ea6ad..116df90 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -126,6 +126,11 @@ JH/25 Fix use of concurrent TLS connections under GnuTLS. When a callout was used rather than per-connection info for tracking the state of data queued for transmission. This could result in a connection hang. +JH/26 Fix use of the SIZE parameter on MAIL commands, on continued connections. + Previously, when delivering serveral messages down a single connection + only the first would provide a SIZE. This was due to the size information + not being properly tracked. + Exim version 4.93 ----------------- diff --git a/src/src/transports/smtp.c b/src/src/transports/smtp.c index b748b76..5c5f37d 100644 --- a/src/src/transports/smtp.c +++ b/src/src/transports/smtp.c @@ -2377,6 +2377,7 @@ else } sx->inblock.cctx = sx->outblock.cctx = &sx->cctx; smtp_command = big_buffer; + sx->peer_offered = smtp_peer_options; sx->helo_data = NULL; /* ensure we re-expand ob->helo_data */ /* For a continued connection with TLS being proxied for us, or a @@ -2387,7 +2388,6 @@ else && cutthrough.is_tls) ) { - sx->peer_offered = smtp_peer_options; sx->pipelining_used = pipelining_active = !!(smtp_peer_options & OPTION_PIPE); HDEBUG(D_transport) debug_printf("continued connection, %s TLS\n", continue_proxy_cipher ? "proxied" : "verify conn with"); @@ -3512,7 +3512,6 @@ always has a sequence number greater than one. */ if (continue_hostname && continue_sequence == 1) { - sx->peer_offered = smtp_peer_options; /* sx->pending_MAIL = FALSE; */ sx->ok = TRUE; /* sx->next_addr = NULL; */ -- You are receiving this mail because: You are on the CC list for the bug. -- ## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##
