tree a7d2263b0adc5ee4f7dfb57d04a6f54a6d67839b
parent 5a9c875f0f1ee83bd5889dd1ad53e9da43e6c34e
author Olivier Houchard <ohouchard@haproxy.com> 1564589126 +0200
committer Willy Tarreau <w@1wt.eu> 1565081600 +0200
encoding latin1

BUG/MEDIUM: proxy: Make sure to destroy the stream on upgrade from TCP to H2

In stream_set_backend(), if we have a TCP stream, and we want to upgrade it
to H2 instead of attempting ot reuse the stream, just destroy the
conn_stream, make sure we don't log anything about the stream, and pretend
we failed setting the backend, so that the stream will get destroyed.
New streams will then be created by the mux, as if the connection just
happened.
This fixes a crash when upgrading from TCP to H2, as the H2 mux totally
ignored the conn_stream provided by the upgrade, as reported in github
issue #196.

This should be backported to 2.0.

(cherry picked from commit 4c18f94c11b11e4a4923892ed0fa72385ec1eb08)
Signed-off-by: Willy Tarreau <w@1wt.eu>