PP/12 fix uninitialised greeting string from PP/03 exim-4_77_RC1
authorPhil Pennock <pdp@exim.org>
Mon, 3 Oct 2011 12:10:11 +0000 (08:10 -0400)
committerPhil Pennock <pdp@exim.org>
Mon, 3 Oct 2011 12:10:11 +0000 (08:10 -0400)
doc/doc-txt/ChangeLog
src/src/transports/smtp.c

index a911d80..b58444a 100644 (file)
@@ -121,6 +121,8 @@ PP/11 match_* no longer expand right-hand-side by default.
       New compile-time build option, EXPAND_LISTMATCH_RHS.
       New expansion conditions, "inlist", "inlisti".
 
+PP/12 fix uninitialised greeting string from PP/03 (smtps client support).
+
 
 Exim version 4.76
 -----------------
index 53012ec..c571d87 100644 (file)
@@ -1181,21 +1181,20 @@ if (tls_active >= 0)
       }
     }
 
-  /* For SMTPS we need to wait for the initial OK response.
-  Also, it seems likely that a server not supporting STARTTLS is broken
-  enough to perhaps not support EHLO. */
+  /* For SMTPS we need to wait for the initial OK response. */
   if (smtps)
     {
     if (!smtp_read_response(&inblock, buffer, sizeof(buffer), '2',
       ob->command_timeout)) goto RESPONSE_FAILED;
-    if (esmtp)
-      greeting_cmd = "EHLO";
-    else
-      {
-      greeting_cmd = "HELO";
-      DEBUG(D_transport)
-        debug_printf("not sending EHLO (host matches hosts_avoid_esmtp)\n");
-      }
+    }
+
+  if (esmtp)
+    greeting_cmd = "EHLO";
+  else
+    {
+    greeting_cmd = "HELO";
+    DEBUG(D_transport)
+      debug_printf("not sending EHLO (host matches hosts_avoid_esmtp)\n");
     }
 
   if (smtp_write_command(&outblock, FALSE, "%s %s\r\n",