X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fsrc%2Fsmtp_in.c;h=00c842760525b1e49632ab71bc5c78071132d1de;hb=ce80533b305c56d57cb7ec1484491f191132cf84;hp=55272a99a0ebbbbe153262cf6a391ddb841dbc43;hpb=88752192a16ea6f8f8bf4d3b3f801e20d49e5398;p=exim.git

diff --git a/src/src/smtp_in.c b/src/src/smtp_in.c
index 55272a99a..00c842760 100644
--- a/src/src/smtp_in.c
+++ b/src/src/smtp_in.c
@@ -2367,6 +2367,7 @@ return done - 2;  /* Convert yield values */
 
 
 
+#ifdef SUPPORT_TLS
 static BOOL
 smtp_log_tls_fail(uschar * errstr)
 {
@@ -2378,6 +2379,7 @@ if (Ustrncmp(conn_info, US"SMTP ", 5) == 0) conn_info += 5;
 log_write(0, LOG_MAIN, "TLS error on %s %s", conn_info, errstr);
 return FALSE;
 }
+#endif
 
 
 
@@ -3907,7 +3909,7 @@ while (done <= 0)
   int c;
   auth_instance *au;
   uschar *orcpt = NULL;
-  int flags;
+  int dsn_flags;
   gstring * g;
 
 #ifdef AUTH_TLS
@@ -4960,7 +4962,7 @@ while (done <= 0)
 
       /* Set the DSN flags orcpt and dsn_flags from the session*/
       orcpt = NULL;
-      flags = 0;
+      dsn_flags = 0;
 
       if (fl.esmtp) for(;;)
 	{
@@ -4985,14 +4987,14 @@ while (done <= 0)
 	else if (fl.dsn_advertised && strcmpic(name, US"NOTIFY") == 0)
 	  {
 	  /* Check if the notify flags have been already set */
-	  if (flags > 0)
+	  if (dsn_flags > 0)
 	    {
 	    done = synprot_error(L_smtp_syntax_error, 501, NULL,
 		US"NOTIFY can be specified once only");
 	    goto COMMAND_LOOP;
 	    }
 	  if (strcmpic(value, US"NEVER") == 0)
-	    flags |= rf_notify_never;
+	    dsn_flags |= rf_notify_never;
 	  else
 	    {
 	    uschar *p = value;
@@ -5004,17 +5006,17 @@ while (done <= 0)
 	      if (strcmpic(p, US"SUCCESS") == 0)
 		{
 		DEBUG(D_receive) debug_printf("DSN: Setting notify success\n");
-		flags |= rf_notify_success;
+		dsn_flags |= rf_notify_success;
 		}
 	      else if (strcmpic(p, US"FAILURE") == 0)
 		{
 		DEBUG(D_receive) debug_printf("DSN: Setting notify failure\n");
-		flags |= rf_notify_failure;
+		dsn_flags |= rf_notify_failure;
 		}
 	      else if (strcmpic(p, US"DELAY") == 0)
 		{
 		DEBUG(D_receive) debug_printf("DSN: Setting notify delay\n");
-		flags |= rf_notify_delay;
+		dsn_flags |= rf_notify_delay;
 		}
 	      else
 		{
@@ -5025,7 +5027,7 @@ while (done <= 0)
 		}
 	      p = pp;
 	      }
-	      DEBUG(D_receive) debug_printf("DSN Flags: %x\n", flags);
+	      DEBUG(D_receive) debug_printf("DSN Flags: %x\n", dsn_flags);
 	    }
 	  }
 
@@ -5143,7 +5145,7 @@ while (done <= 0)
 
 	/* Set the dsn flags in the recipients_list */
 	recipients_list[recipients_count-1].orcpt = orcpt;
-	recipients_list[recipients_count-1].dsn_flags = flags;
+	recipients_list[recipients_count-1].dsn_flags = dsn_flags;
 
 	DEBUG(D_receive) debug_printf("DSN: orcpt: %s  flags: %d\n",
 	  recipients_list[recipients_count-1].orcpt,
@@ -5506,7 +5508,9 @@ while (done <= 0)
 
       /* Hard failure. Reject everything except QUIT or closed connection. One
       cause for failure is a nested STARTTLS, in which case tls_in.active remains
-      set, but we must still reject all incoming commands. */
+      set, but we must still reject all incoming commands.  Another is a handshake
+      failure - and there may some encrypted data still in the pipe to us, which we
+      see as garbage commands. */
 
       DEBUG(D_tls) debug_printf("TLS failed to start\n");
       while (done <= 0) switch(smtp_read_command(FALSE, GETC_BUFFER_UNLIMITED))