From 833c70bc6dc0b85ea942bb127515d582a91a6ed7 Mon Sep 17 00:00:00 2001 From: Phil Pennock Date: Mon, 26 Jun 2017 18:39:29 -0400 Subject: Repair manualroute transport name not-last option Commit 55414b25b did const-ification surgery which removed the old "temporarily drop a NUL in here" anti-pattern, but one of the Ustrcmp->Ustrncmp conversions was missed, for transport-name comparison. As a result, since Exim 4.85, we've had an undocumented requirement-by-bug that the transport name be the last such option given. Restore old behaviour. Still missing a test-case to deter regression. --- doc/doc-docbook/spec.xfpt | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'doc/doc-docbook') diff --git a/doc/doc-docbook/spec.xfpt b/doc/doc-docbook/spec.xfpt index 4a239b73c..aab8c00d5 100644 --- a/doc/doc-docbook/spec.xfpt +++ b/doc/doc-docbook/spec.xfpt @@ -19469,6 +19469,10 @@ instead of TRY_AGAIN. That is why the default action is to try a DNS lookup first. Only if that gives a definite &"no such host"& is the local function called. +&*Compatibility*&: From Exim 4.85 until fixed for 4.90, there was an +inadvertent constraint that a transport name as an option had to be the last +option specified. + If no IP address for a host can be found, what happens is controlled by the -- cgit v1.2.3