BUG/MAJOR: stream: in stream_free(), close the front endpoint and not the origin
authorWilly Tarreau <w@1wt.eu>
Thu, 17 Aug 2017 13:54:46 +0000 (15:54 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 17 Aug 2017 16:26:56 +0000 (18:26 +0200)
commit7632548d9789e1588a1ad70ced3b87ef06d41b21
treecd25c37bd12f6b569cfe1b3339092c2d13927bc5
parent50bc31d4d6a6861eac3ca2298b93cd62a51310ed
BUG/MAJOR: stream: in stream_free(), close the front endpoint and not the origin

stream_free() used to close the front connection by using s->sess->origin,
instead of using s->si[0].end. This is very visible in HTTP/2 where the
front connection is abusively closed and causes all sort of issues including
crashes caused by double closes due to the same origin being referenced many
times.

It's also suspected that it may have caused some of the early issues met
during the Lua development.

It's uncertain whether stable branches are affected. It might be worth
backporting it once it has been confirmed not to create new impacts.
src/stream.c