X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fsrc%2Fsmtp_in.c;h=86f87eae1e406d02bb593ecb95b736e15a2662b9;hb=refs%2Ftags%2Fexim-4.92;hp=54ebf3660892023eef45bcf9b0f4bbee28ada334;hpb=d12746bc15d83ab821be36975da0179672708bc1;p=user%2Fhenk%2Fcode%2Fexim.git

diff --git a/src/src/smtp_in.c b/src/src/smtp_in.c
index 54ebf3660..86f87eae1 100644
--- a/src/src/smtp_in.c
+++ b/src/src/smtp_in.c
@@ -1777,7 +1777,7 @@ if (f.sender_host_unknown || f.sender_host_notsocket)
 if (f.is_inetd)
   return string_sprintf("SMTP connection from %s (via inetd)", hostname);
 
-if (LOGGING(incoming_interface) && interface_address != NULL)
+if (LOGGING(incoming_interface) && interface_address)
   return string_sprintf("SMTP connection from %s I=[%s]:%d", hostname,
     interface_address, interface_port);
 
@@ -2084,7 +2084,6 @@ f.dkim_disable_verify = FALSE;
 dkim_collect_input = 0;
 dkim_verify_overall = dkim_verify_status = dkim_verify_reason = NULL;
 dkim_key_length = 0;
-dkim_verify_signers = US"$dkim_signers";
 #endif
 #ifdef EXPERIMENTAL_DMARC
 f.dmarc_has_been_checked = f.dmarc_disable_verify = f.dmarc_enable_forensic = FALSE;
@@ -3485,7 +3484,7 @@ int rc;
 uschar *user_msg = NULL;
 uschar *log_msg = NULL;
 
-/* Check for recursive acll */
+/* Check for recursive call */
 
 if (fl.smtp_exit_function_called)
   {
@@ -3510,9 +3509,7 @@ if (acl_smtp_notquit && reason)
 tls_in.active.sock = -1;
 
 /* Write an SMTP response if we are expected to give one. As the default
-responses are all internal, they should always fit in the buffer, but code a
-warning, just in case. Note that string_vformat() still leaves a complete
-string, even if it is incomplete. */
+responses are all internal, they should be reasonable size. */
 
 if (code && defaultrespond)
   {
@@ -4171,9 +4168,8 @@ while (done <= 0)
 	because otherwise the log can be confusing. */
 
 	if (  !sender_host_name
-	   && (deliver_domain = sender_helo_name,  /* set $domain */
-	       match_isinlist(sender_helo_name, CUSS &helo_lookup_domains, 0,
-		&domainlist_anchor, NULL, MCL_DOMAIN, TRUE, NULL)) == OK)
+	   && match_isinlist(sender_helo_name, CUSS &helo_lookup_domains, 0,
+		&domainlist_anchor, NULL, MCL_DOMAIN, TRUE, NULL) == OK)
 	  (void)host_name_lookup();
 
 	/* Rebuild the fullhost info to include the HELO name (and the real name
@@ -4262,20 +4258,15 @@ while (done <= 0)
       smtp_code = US"250 ";        /* Default response code plus space*/
       if (!user_msg)
 	{
-	s = string_sprintf("%.3s %s Hello %s%s%s",
+	g = string_fmt_append(NULL, "%.3s %s Hello %s%s%s",
 	  smtp_code,
 	  smtp_active_hostname,
 	  sender_ident ? sender_ident : US"",
 	  sender_ident ? US" at " : US"",
 	  sender_host_name ? sender_host_name : sender_helo_name);
-	g = string_cat(NULL, s);
 
 	if (sender_host_address)
-	  {
-	  g = string_catn(g, US" [", 2);
-	  g = string_cat (g, sender_host_address);
-	  g = string_catn(g, US"]", 1);
-	  }
+	  g = string_fmt_append(g, " [%s]", sender_host_address);
 	}
 
       /* A user-supplied EHLO greeting may not contain more than one line. Note
@@ -4313,11 +4304,8 @@ while (done <= 0)
 	till then, VRFY and EXPN can be used after EHLO when space is short. */
 
 	if (thismessage_size_limit > 0)
-	  {
-	  sprintf(CS big_buffer, "%.3s-SIZE %d\r\n", smtp_code,
+	  g = string_fmt_append(g, "%.3s-SIZE %d\r\n", smtp_code,
 	    thismessage_size_limit);
-	  g = string_cat(g, big_buffer);
-	  }
 	else
 	  {
 	  g = string_catn(g, smtp_code, 3);