tree 48647fc4667b864b91e194dde33ffcf5f3e2f3cb
parent ab1300b6eeb85412fc569c569cceba3bed0b86e7
author Olivier Houchard <ohouchard@haproxy.com> 1562596232 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1562596689 +0200

BUG/MEDIUM: servers: Don't forget to set srv_cs to NULL if we can't reuse it.

In connect_server(), if there were already a CS assosciated with the stream,
but we can't reuse it, because the target is different (because we tried a
previous connection, it failed, and we use redispatch so we switched servers),
don't forget to set srv_cs to NULL. Otherwise, if we end up reusing another
connection, we would consider we already have a conn_stream, and we won't
create a new one, so we'd have a new connection but we would not be able to
use it.
This can explain frozen streams and connections stuck in CLOSE_WAIT when
using redispatch.

This should be backported to 1.9 and 2.0.

(cherry picked from commit a1ab97316f34dc3b478cd076b76b01c0e23b8769)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>