bug-fix test-driving input
authorPhil Pennock <pdp@exim.org>
Wed, 1 Feb 2017 17:34:52 +0000 (12:34 -0500)
committerPhil Pennock <pdp@exim.org>
Wed, 1 Feb 2017 17:34:52 +0000 (12:34 -0500)
The client driver is a little restrictive in the escape sequences it
handles; two octets here were missing the `x` after the `\`, so `\05` is
two octets, a 0 and then a 5, where `\x05` would be one octet.

So we were sending two more octets than expected, not catching that Exim
was parsing the wrong IP/port at the end, and now that Exim only reads
as much of the proxy protocol header as belongs in it, instead of "up to
the largest it could be", this test-bug has been exposed.

test/scripts/4030-proxy-protocol/4030

index 8b560bcbcc7b71e6dec4fe5c9f1a54d087fd01c9..50dca160a0fcef20beedca89279ab1e646370550 100644 (file)
@@ -50,7 +50,7 @@ QUIT
 #
 # protocol v2 plain receive
 client HOSTIPV4 PORT_D
->>> \x0D\x0A\x0D\x0A\x00\x0D\x0A\x51\x55\x49\x54\x0A\x21\x11\x00\x0c\xc0\xa8\x00\x0f\xc0\xa8\x00\05\xc2\x95\x04\01
+>>> \x0D\x0A\x0D\x0A\x00\x0D\x0A\x51\x55\x49\x54\x0A\x21\x11\x00\x0c\xc0\xa8\x00\x0f\xc0\xa8\x00\x05\xc2\x95\x04\x01
 ??? 220
 HELO clientname
 ??? 250