diff options
| author | Phil Pennock <pdp@exim.org> | 2017-06-26 18:39:29 -0400 |
|---|---|---|
| committer | Phil Pennock <pdp@exim.org> | 2017-06-26 18:44:33 -0400 |
| commit | 833c70bc6dc0b85ea942bb127515d582a91a6ed7 (patch) | |
| tree | 903a5bb15d683c5b244f60c2af68d72023b8727f /src | |
| parent | af542dcfa62fc2341e8253f10c5c778b887444b9 (diff) | |
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.
Diffstat (limited to 'src')
| -rw-r--r-- | src/src/routers/manualroute.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/src/routers/manualroute.c b/src/src/routers/manualroute.c index 95c69328d..31185e160 100644 --- a/src/src/routers/manualroute.c +++ b/src/src/routers/manualroute.c @@ -333,7 +333,7 @@ while (*options != 0) { transport_instance *t; for (t = transports; t != NULL; t = t->next) - if (Ustrcmp(t->name, s) == 0) + if (Ustrncmp(t->name, s, n) == 0) { transport = t; individual_transport_set = TRUE; |