From: Wilfried Goesgens <dothebart@citadel.org>
Date: Thu, 28 Apr 2011 19:10:56 +0000 (+0000)
Subject: Name of MX in error messages
X-Git-Tag: v8.11~1075
X-Git-Url: https://code.citadel.org/?p=citadel.git;a=commitdiff_plain;h=3efed78f36bb50cf280d802f438f94319f4cad45

Name of MX in error messages

  - set it after successfull MX lookup
  - put in that we don't have an mx record if its NULL instead of 'no name'
---

diff --git a/citadel/modules/smtp/serv_smtpeventclient.c b/citadel/modules/smtp/serv_smtpeventclient.c
index 85ef2813b..fcec898cb 100644
--- a/citadel/modules/smtp/serv_smtpeventclient.c
+++ b/citadel/modules/smtp/serv_smtpeventclient.c
@@ -207,7 +207,7 @@ void SetConnectStatus(AsyncIO *IO)
 		  sizeof(buf));
 
 	if (SendMsg->mx_host == NULL)
-		SendMsg->mx_host = "<no name>";
+		SendMsg->mx_host = "<no MX-Record>";
 
 	CtdlLogPrintf(CTDL_DEBUG, 
 		      "SMTP client[%ld]: connecting to %s [%s]:%d ...\n", 
@@ -275,6 +275,7 @@ eNextState get_one_mx_host_ip_done(AsyncIO *IO)
 			addr->sin_port   = htons(DefaultMXPort);
 			
 		}
+		SendMsg->mx_host = SendMsg->pCurrRelay->Host;
 		return mx_connect_ip(IO);
 	}
 	else // TODO: here we need to find out whether there are more mx'es, backup relay, and so on