diff options
| author | Tony Finch <dot@dot.at> | 2008-09-05 16:59:47 +0000 |
|---|---|---|
| committer | Tony Finch <dot@dot.at> | 2008-09-05 16:59:47 +0000 |
| commit | e276e04bb6468a6b9f5d0f67cc84b1921c6020d9 (patch) | |
| tree | 135c47c218db9be94c41d895cea120d13fe99f6d /src | |
| parent | 7199e1ee2c175060f708ff91ccaf992b5afdbaa9 (diff) | |
Produce a more useful error message if an SMTP transport's hosts
setting expands to an empty string.
Bug reported by Jens Hoffrichter <jens.hoffrichter@gmail.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/src/transports/smtp.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/src/transports/smtp.c b/src/src/transports/smtp.c index 4e1f3c45b..e8354ad7a 100644 --- a/src/src/transports/smtp.c +++ b/src/src/transports/smtp.c @@ -1,4 +1,4 @@ -/* $Cambridge: exim/src/src/transports/smtp.c,v 1.39 2008/03/05 21:13:23 tom Exp $ */ +/* $Cambridge: exim/src/src/transports/smtp.c,v 1.40 2008/09/05 16:59:48 fanf2 Exp $ */ /************************************************* * Exim - an Internet mail transport agent * @@ -2264,6 +2264,15 @@ if (hostlist == NULL || (ob->hosts_override && ob->hosts != NULL)) host_build_hostlist(&hostlist, s, ob->hosts_randomize); + /* Check that the expansion yielded something useful. */ + if (hostlist == NULL) + { + addrlist->message = + string_sprintf("%s transport has empty hosts setting", tblock->name); + addrlist->transport_return = PANIC; + return FALSE; /* Only top address has status */ + } + /* If there was no expansion of hosts, save the host list for next time. */ |