Repair manualroute transport name not-last option
authorPhil Pennock <pdp@exim.org>
Mon, 26 Jun 2017 22:39:29 +0000 (18:39 -0400)
committerPhil Pennock <pdp@exim.org>
Mon, 26 Jun 2017 22:44:33 +0000 (18:44 -0400)
commit833c70bc6dc0b85ea942bb127515d582a91a6ed7
tree903a5bb15d683c5b244f60c2af68d72023b8727f
parentaf542dcfa62fc2341e8253f10c5c778b887444b9
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
doc/doc-txt/ChangeLog
src/src/routers/manualroute.c