From e6e854a44d877408fbe8e55fd71abb1c40150d3c Mon Sep 17 00:00:00 2001
From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Sat, 22 Oct 2016 00:41:44 +0100
Subject: [PATCH] Move smtp_init's last-ditch attempt for client inside first
 test.

No point checking it twice in a row if it was false the first time.
---
 mts/smtp/smtp.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/mts/smtp/smtp.c b/mts/smtp/smtp.c
index f4308ba5..7b706e01 100644
--- a/mts/smtp/smtp.c
+++ b/mts/smtp/smtp.c
@@ -128,14 +128,13 @@ smtp_init (char *client, char *server, char *port, int watch, int verbose,
 	} else {
 	    client = LocalName(1);	/* no clientname -> LocalName */
 	}
-    }
-
-    /*
-     * Last-ditch check just in case client still isn't set to anything
-     */
 
-    if (client == NULL || *client == '\0')
-	client = "localhost";
+        /*
+         * Last-ditch check just in case client still isn't set to anything
+         */
+        if (client == NULL || *client == '\0')
+            client = "localhost";
+    }
 
     nsc = netsec_init();
 
-- 
2.56.0