From d41e4a9122eb42ec4fdc3bc6cc2a5094d0e515f1 Mon Sep 17 00:00:00 2001
From: Tom Kistner <tom@duncanthrax.net>
Date: Mon, 19 Jul 2010 11:47:27 +0200
Subject: [PATCH] Bugzilla #1006: Keep EHLO attributes in case STARTTLS errors
 are ignored

Applied patch submitted by Micha Lenk. Thanks!
---
 src/src/transports/smtp.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/src/transports/smtp.c b/src/src/transports/smtp.c
index fb55ae01b..41796a4d9 100644
--- a/src/src/transports/smtp.c
+++ b/src/src/transports/smtp.c
@@ -1078,10 +1078,12 @@ if (tls_offered && !suppress_tls &&
   if (!smtp_read_response(&inblock, buffer2, sizeof(buffer2), '2',
       ob->command_timeout))
     {
-    Ustrncpy(buffer, buffer2, sizeof(buffer));
     if (errno != 0 || buffer2[0] == 0 ||
          (buffer2[0] == '4' && !ob->tls_tempfail_tryclear))
+      {
+      Ustrncpy(buffer, buffer2, sizeof(buffer));
       goto RESPONSE_FAILED;
+      }
     }
 
   /* STARTTLS accepted: try to negotiate a TLS session. */
-- 
2.30.2